[m-rev.] for review: bulk unify and compare
Julien Fischer
jfischer at opturion.com
Thu Oct 4 00:06:00 AEST 2018
On Wed, 3 Oct 2018, Julien Fischer wrote:
> On Wed, 3 Oct 2018, Julien Fischer wrote:
>>
>> On Wed, 3 Oct 2018, Zoltan Somogyi wrote:
>>
>>> I just committed the diff with the changes that the reviews asked for.
>>> It has clean bootchecks in hlc.gc and asm_fast.gc, but there were
>>> *lots* of differences from tools/make_java_csharp_diff; too many
>>> to visually check them all. The ones I did check were all trivial
>>> changes
>>> in variable names, a consequence of the fact that unify_proc.m now
>>> allocates variables in a different order than it used to, which changes
>>> HLDS variable numbers (which get included in Java and C# variable
>>> names).
>>>
>>> Julien, can I ask you to try out the updated compiler in the Java and C#
>>> grades?
>>
>> I've successfully bootchecked the updated compiler in the Java grade;
>> the C# bootcheck is running now.
>
> The C# grade also succesfully bootchecked.
It has however broken the Erlang grade:
...
Making Mercury/ints/thread.mvar.int
Making Mercury/ints/thread.semaphore.int
Making Mercury/opts/array.opt
Uncaught Mercury exception:
Software Error: function
`check_hlds.unify_proc.compute_maybe_packable_args_locn'/1: Unexpected:
wrong tag for a general du functor with args
Stack dump not available in this grade.
** Error making `Mercury/opts/array.opt'.
Julien.
More information about the reviews
mailing list