[m-rev.] for review: Don't cast int32/uint32 to MR_Word in foreign code input assignments.

Julien Fischer jfischer at opturion.com
Thu Oct 1 14:29:45 AEST 2026


On Thu, 1 Oct 2026 at 14:14, Peter Wang <novalazy at gmail.com> wrote:
>
> This fixes gcc warnings on 64-bit platforms of the form:
>
>     warning: overflow in conversion from ‘long long unsigned int’ to
>     ‘int32_t’ {aka ‘int’} changes value from ‘18446744071562067968’ to
>     ‘-2147483648’ [-Woverflow]
>
> for the generated code:
>
>     I32 = ((MR_Word) INT32_MIN);
>
> compiler/llds_out_instr.m:
>     In output_foreign_proc_inputs, output int32/uint32 input values
>     without a cast to MR_Word.

That's fine -- thanks for fixing that.

Julien.


More information about the reviews mailing list