[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