Re: [PATCH] Fix IPA-ICF wrong-code with GIMPLE_PREDICT (PR ipa/65765)

2015-04-15 Thread Jan Hubicka
> Hi! > > When ICF encounteres a GIMPLE_PREDICT (as in the testcase) or GIMPLE_NOP, > it stops looking at further statements in the bb (assumes they must be the > same), which is of course wrong. > > I'm attaching 3 versions of the patch, the first one I've > bootstrapped/regtested on x86_64-linu

Re: [PATCH] Fix IPA-ICF wrong-code with GIMPLE_PREDICT (PR ipa/65765)

2015-04-15 Thread Richard Biener
On Wed, 15 Apr 2015, Jakub Jelinek wrote: > Hi! > > When ICF encounteres a GIMPLE_PREDICT (as in the testcase) or GIMPLE_NOP, > it stops looking at further statements in the bb (assumes they must be the > same), which is of course wrong. > > I'm attaching 3 versions of the patch, the first one I

[PATCH] Fix IPA-ICF wrong-code with GIMPLE_PREDICT (PR ipa/65765)

2015-04-15 Thread Jakub Jelinek
Hi! When ICF encounteres a GIMPLE_PREDICT (as in the testcase) or GIMPLE_NOP, it stops looking at further statements in the bb (assumes they must be the same), which is of course wrong. I'm attaching 3 versions of the patch, the first one I've bootstrapped/regtested on x86_64-linux and i686-linux