[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