[m-rev.] for preliminary review: bug 493 & 532 alterative fix

Zoltan Somogyi zoltan.somogyi at runbox.com
Sat May 15 13:41:10 AEST 2021


2021-05-15 12:47 GMT+10:00 "Peter Wang" <novalazy at gmail.com>:
>> >     --- a/a/stage2/compiler/hlds.make_hlds.state_var.c
>> >     +++ b/b/stage2/compiler/hlds.make_hlds.state_var.c
>> >     ...
>> >     @@ -1263,12 +1263,6 @@ static /* final */ const MR_Box hlds__make_hlds__state_var_scalar_common_2[6][3]
>> > 	 ((MR_Box) ((MR_Unsigned) 0U)),
>> > 	 ((MR_Box) ((MR_Unsigned) 0U))
>> >        },
>> >     -  /* row 5 */
>> >     -  {
>> >     -    ((MR_Box) (((MR_Box) ((MR_Integer) 1)))),
>> >     -    ((MR_Box) ((MR_Unsigned) 0U)),
>> >     -    ((MR_Box) ((MR_Unsigned) 0U))
>> >     -  },
>> >      };
>> 
>> What options did you use to generate the code in this diff?
> 
> % cat Mmake.stage.params
> GRADE=hlc.gc
> EXTRA_MCFLAGS=-O5 --intermod-opt

When I do that, I get hlds__make_hlds__state_var_scalar_common_2
ending after row 4; no row 5 at all. Row 4 does have the same content
as above.

Zoltan.


More information about the reviews mailing list