[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