"Attila Lendvai" <attila.lendvai@...> writes:
> i thought it's only due to a small mistake, so i'm replying to the
> list now if you don't mind.
>
> i personally like my code loading/compiling without warnings and i
> keep a sharp eye for new compiler messages at each recompile. from
> this point of view, that note is a noise on normal optimization levels
> which make it harder to catch new notes.
For what it's worth, I think I agree with you here: the only
:important transforms prior to this were those involving running the
compiler at runtime -- which is still a judgment call -- but also only
in one subsystem (the alien FFI), so that people writing ordinary lisp
code without any particular optimization settings don't get compiler
notes. I think that's a reasonable default, and I don't see a 5x
speedup for one operator being a desperately good reason to bother the
user at default settings. (If it were something like "unless you fix
this, you can't dump a compilerless core", or something similarly
qualitatively different, then I think that would be a sufficiently
good reason to notify the user at default settings, but not just for a
speedup: after all, the user might in the end prefer space savings).
(I should say that I'm not completely wild about having both the
compiler macro and the transforms, at least in source form -- I don't
mind how it's implemented under the hood if having both leads to a
measureable speedup in compile time, but the duplication of logic (and
";; see comment in the compiler-macro" :-) suggest that the transform
and the compiler macros should really be only implemented once in the
source.)
> not to mention those unused .ARG42. warnings coming from PCL. but i
> should cut that out in antoher mail.
Yes please.
Cheers,
Christophe
|