[m-rev.] for testing: list all may-introduce-calls-to predicates
Julien Fischer
jfischer at opturion.com
Wed Feb 23 20:56:52 AEDT 2022
Hi Zoltan,
On Sun, 20 Feb 2022, Julien Fischer wrote:
> On Thu, 17 Feb 2022, Zoltan Somogyi wrote:
>
>> This diff affects only one file, but it rewrites that file almost
>> completely, so I am sending the file instead of a diff.
>>
>> There are some XXXs that may be worth reading,
>> but the rest of the file cannot be effectively reviewed
>> by humans, because what's there cannot be the cause
>> of a bug; bugs can be caused only by stuff being *missing*.
>>
>> I tracked down the predicates and functions we may introduce
>> calls to by looking for plain_call and call_foreign_proc goals
>> being constructed, and tracking where their pred_ids came from.
>> If a pred_id came from the lookup of the name of a predicate
>> or of a function, I included them in the list. However, since
>> it is possible that I may have missed some, this diff should
>> be tested thoroughly. Julien, can you please exercise
>> a compiler with this diff with all the grade/optlevel combinations
>> you tested the release with on your -j16 machine? That is
>> the "review" that this diff needs.
>
> The stage 2 library fails to compile when bootchecking in grade
> asm_fast.gc.profdeep.stseg.
>
> Uncaught Mercury exception:
> Software Error: predicate
> `hlds.pred_table.lookup_builtin_pred_proc_id'/8: Unexpected: can't
> locate builtin.compare_representation/3
> Stack dump not available in this grade.
>
> (FYI: the change has so far bootstapped at all optimization levels, with
> and without --intermod-opt, in the following grades: asm_fast.gc,
> hlc.gc, asm_fast.gc.tr, hlc.gc.tr, asm_fast.gc.debug.stseg and
> asm_fast.gc.decldebug.stseg.)
Further bootchecks at all optimization levels have been completed for
grade asm_fast.gc.prof, asm_fast.gc.memprof, asm_fast.gc.mm.stseg,
hlc.gc.prof, hlc.gc.memprof and hlc.gc.pregen. I've also carried out
bootchecks in the java and csharp grade for a number of optimization
levels.
The only issue with this change is the issue I reported above in
the deep profiling grades.
Julien.
More information about the reviews
mailing list