[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