[m-rev.] for review: pack sub-word-sized integers and dummies
Julien Fischer
jfischer at opturion.com
Sun May 6 14:25:23 AEST 2018
On Sat, 5 May 2018, Zoltan Somogyi wrote:
>
>
> On Sun, 22 Apr 2018 06:05:33 +1000 (AEST), "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
>>> Looks fine otherwise.
>>
>> Thanks for the review.
>
> The new testing I added for this review revealed a couple of problems,
> which I have now fixed, so I am now committing the attached updated diff.
> I have bootchecked it twelve times:
>
> - in hlc.gc, asm_fast.gc and asm_fast.debug.stseg.gc;
> - in 64 and 32 bit mode; and
> - with the new functionality switched on and off.
Building the library in the C# grade now breaks:
Making Mercury/opts/thread.semaphore.opt
Making Mercury/css/array.cs
Uncaught Mercury exception:
Software Error: predicate `ml_backend.ml_unify_gen.ml_gen_cons_args'/12: Unexpected: full
Stack dump not available in this grade.
** Error making `Mercury/css/array.cs'.
Julien.
More information about the reviews
mailing list