Re: [PATCH] Fix loop pattern distribution ICE (PR tree-optimization/54321)

2012-08-20 Thread Richard Guenther
On Mon, Aug 20, 2012 at 8:27 AM, Jakub Jelinek wrote: > Hi! > > The middle-end argument of memset is signed (int), so simplify_builtin_call > correctly checks host_integerp (val2, 0), but later on used tree_low_cst > (val2, 1), so for negative values it would ICE. Fixed thusly, the memset > is su

[PATCH] Fix loop pattern distribution ICE (PR tree-optimization/54321)

2012-08-19 Thread Jakub Jelinek
Hi! The middle-end argument of memset is signed (int), so simplify_builtin_call correctly checks host_integerp (val2, 0), but later on used tree_low_cst (val2, 1), so for negative values it would ICE. Fixed thusly, the memset is supposed to cast the int to unsigned char internally anyway. Bootst