Re: [PATCH] Fix undefined behavior in the combiner

2014-03-25 Thread Richard Biener
On Tue, 25 Mar 2014, Jakub Jelinek wrote: > Hi! > > simplify_compare_const is sometimes called with two CONST_INTs, > which means mode is VOIDmode and mode_width is 0, but then it > happily verifies that mode_width is <= HOST_BITS_PER_WIDE_INT > and shits stuff by mode_width - 1 (i.e. -1). > From

[PATCH] Fix undefined behavior in the combiner

2014-03-25 Thread Jakub Jelinek
Hi! simplify_compare_const is sometimes called with two CONST_INTs, which means mode is VOIDmode and mode_width is 0, but then it happily verifies that mode_width is <= HOST_BITS_PER_WIDE_INT and shits stuff by mode_width - 1 (i.e. -1). >From what I can understand, some optimizations can be still