Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2015-01-09 Thread Richard Biener
On Mon, Jan 5, 2015 at 2:12 PM, Martin Liška wrote: > On 12/19/2014 12:04 PM, Richard Biener wrote: >> >> On Thu, Dec 18, 2014 at 6:38 PM, Martin Liška wrote: >>> >>> On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: > > >>

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2015-01-05 Thread Martin Liška
On 12/19/2014 12:04 PM, Richard Biener wrote: On Thu, Dec 18, 2014 at 6:38 PM, Martin Liška wrote: On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Li

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-19 Thread Richard Biener
On Thu, Dec 18, 2014 at 6:38 PM, Martin Liška wrote: > On 12/17/2014 04:23 PM, Richard Biener wrote: >> >> On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: >>> >>> On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: > > >

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-19 Thread Richard Biener
On Thu, Dec 18, 2014 at 2:39 PM, Martin Liška wrote: > On 12/17/2014 04:23 PM, Richard Biener wrote: >> >> On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: >>> >>> On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: > > >

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-18 Thread Martin Liška
On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: Hello. As suggested by Richard, I split compare_operand functions to various functions rel

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-18 Thread Martin Liška
On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: Hello. As suggested by Richard, I split compare_operand functions to various functions rel

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-17 Thread Richard Biener
On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška wrote: > On 12/11/2014 01:37 PM, Richard Biener wrote: >> >> On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: >>> >>> Hello. >>> >>> As suggested by Richard, I split compare_operand functions to various >>> functions >>> related to a specific com

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-17 Thread Martin Liška
On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: Hello. As suggested by Richard, I split compare_operand functions to various functions related to a specific comparison. Apart from that I added fast check for volatility flag that caused miscomp

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-11 Thread Richard Biener
On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška wrote: > Hello. > > As suggested by Richard, I split compare_operand functions to various > functions > related to a specific comparison. Apart from that I added fast check for > volatility flag that caused miscompilation mentioned in PR63569. > > Patc

[PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-10 Thread Martin Liška
Hello. As suggested by Richard, I split compare_operand functions to various functions related to a specific comparison. Apart from that I added fast check for volatility flag that caused miscompilation mentioned in PR63569. Patch can bootstrap on x86_64-linux-pc without any regression seen and