[m-rev.] for review: start using type_spec_constrained_preds pragmas
Julien Fischer
jfischer at opturion.com
Fri Feb 9 17:57:53 AEDT 2024
On Fri, 9 Feb 2024, Zoltan Somogyi wrote:
> There are two rotds with the commit that this diff requires.
>
> My laptop's timings are too variable to be able to measure
> the speed impact of this diff. For tools/speedtest, I got a 0.7%
> speedup; for tools/bootcheck -j6, approx the same slowdown.
> If someone with a less volatile machine could try it out,
> I would be grateful.
I will do so. I did try an earlier version of this change out on my JSON
library and got a very small speedup. I have some theories about why
this is, but need to have closer look at what the compiler is doing.
Question: is there a definitive list of what's allowed to appear inside
.opt files somewhere? The list given in the comment at the top of
compiler/intermod.m is incomplete: it doesn't mention typeclasses or
instances for example.
The diff looks fine.
Julien.
More information about the reviews
mailing list