[m-rev.] for post-commit review: fix mantis bug #543

Julien Fischer jfischer at opturion.com
Mon Jan 24 21:08:11 AEDT 2022


On Mon, 24 Jan 2022, Zoltan Somogyi wrote:

> Fix name clash between C modules.
> 
> compiler/mercury_compile_llds_back_end.m:
>     In the usual case of generating low level C code, we generate names
>     for C modules that all have a numeric suffix. When compiling with
>     --everything-in-one-c-function, which is normally enabled only with -O6,
>     we generated C modules whose name did not include a numeric suffix.
>     In the case of three test cases, hard_coded/backend_external*.m,
>     this lack of a suffix caused clashes with the names of hand-written
>     C modules.
>
>     Fix this bug, Mantis #543, by always using numeric suffixes. Ensure this
>     by using the same machinery for generating names both with and without
>     --everything-in-one-c-function.
>
>     Give the predicates involved better names and more natural argument orders.

That's fine.

Julien.


More information about the reviews mailing list