Re: Fix oversight in round_up_loc change

2015-03-04 Thread Eric Botcazou
> There might be a similar issue in the block of code just above: > > wide_int val = value; > bool overflow_p; > > if ((val & (divisor - 1)) == 0) > return value; > > overflow_p = TREE_OVERFLOW (value); > val &= ~(divisor - 1); > > but I don't h

Fix oversight in round_up_loc change

2015-03-03 Thread Eric Botcazou
The fix for PR c/60226 https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=212346 changed the type of the 'divisor' parameter of the round_up_loc function from 'int' to 'unsigned int'. Now the function does the usual masking dance with powers of two, in particular: t = build_int_cst (TRE