Re: [PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-25 Thread Jakub Jelinek
On Tue, Apr 25, 2017 at 06:27:01PM +0200, Marek Polacek wrote: > On Tue, Apr 25, 2017 at 06:09:20PM +0200, Jakub Jelinek wrote: > > On Tue, Apr 25, 2017 at 06:05:25PM +0200, Marek Polacek wrote: > > > Here we are crashing because fold_binary_loc produced a BIT_IOR_EXPR with > > > incompatible opera

Re: [PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-25 Thread Marek Polacek
On Tue, Apr 25, 2017 at 06:09:20PM +0200, Jakub Jelinek wrote: > On Tue, Apr 25, 2017 at 06:05:25PM +0200, Marek Polacek wrote: > > Here we are crashing because fold_binary_loc produced a BIT_IOR_EXPR with > > incompatible operands. Fixed by adding the missing conversion, similarly > > to

Re: [PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-25 Thread Jakub Jelinek
On Tue, Apr 25, 2017 at 06:05:25PM +0200, Marek Polacek wrote: > Here we are crashing because fold_binary_loc produced a BIT_IOR_EXPR with > incompatible operands. Fixed by adding the missing conversion, similarly > to . > > Bootstrapped/r

[PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-25 Thread Marek Polacek
Here we are crashing because fold_binary_loc produced a BIT_IOR_EXPR with incompatible operands. Fixed by adding the missing conversion, similarly to . Bootstrapped/regtested on x86_64-linux, ok for trunk? And 7.1? 2017-04-25 Marek Pola

Re: [PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-11 Thread Richard Biener
On April 11, 2017 10:54:10 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >This is another case where we miss needed folding from argN or their >arguments to the expected expression type (type has to be compatible >with opN's type, but argN is after STRIP_NOPS). > >Bootstrapped/regtested on x86_64-linu

[PATCH] Fix fold_binary_loc BIT_IOR_EXPR folding (PR sanitizer/80349)

2017-04-11 Thread Jakub Jelinek
Hi! This is another case where we miss needed folding from argN or their arguments to the expected expression type (type has to be compatible with opN's type, but argN is after STRIP_NOPS). Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2017-04-11 Jakub Jelinek