Re: [PATCH] Fix PR ipa/77653

2016-09-23 Thread Martin Liška
On 09/23/2016 11:19 AM, Richard Biener wrote: > Ok, but the classpath ones are imported from upstream (and is going to > be removed anyway), so please omit those. > > Richard. Good! I installed following. M. >From 6e23669e0e6572d44939fd0539c438c75c9d381e Mon Sep 17 00:00:00 2001 From: marxin Da

Re: [PATCH] Fix PR ipa/77653

2016-09-23 Thread Richard Biener
On Fri, Sep 23, 2016 at 10:51 AM, Martin Liška wrote: > On 09/22/2016 03:07 PM, David Edelsohn wrote: >> @@ -,13 +2221,11 @@ sem_variable::merge (sem_item *alias_item) >> } >> >>/* We can not merge if address comparsion metters. */ >> - if (original_address_matters && alias_address_

Re: [PATCH] Fix PR ipa/77653

2016-09-23 Thread Martin Liška
On 09/22/2016 03:07 PM, David Edelsohn wrote: > @@ -,13 +2221,11 @@ sem_variable::merge (sem_item *alias_item) > } > >/* We can not merge if address comparsion metters. */ > - if (original_address_matters && alias_address_matters > - && flag_merge_constants < 2) > + if (alias_

Re: [PATCH] Fix PR ipa/77653

2016-09-22 Thread David Edelsohn
@@ -,13 +2221,11 @@ sem_variable::merge (sem_item *alias_item) } /* We can not merge if address comparsion metters. */ - if (original_address_matters && alias_address_matters - && flag_merge_constants < 2) + if (alias_address_matters && flag_merge_constants < 2) { i

Re: [PATCH] Fix PR ipa/77653

2016-09-22 Thread Richard Biener
On Thu, Sep 22, 2016 at 1:18 PM, Martin Liška wrote: > Hi. > > After some investigation, it shows that IPA ICF merges a pair of variables > where > for just one them address matters. Which is obvious error, fixed in attached > patch. > > Patch can bootstrap on ppc64le-redhat-linux and survives r

[PATCH] Fix PR ipa/77653

2016-09-22 Thread Martin Liška
rom d16301adc382a1cf5ede151993633a2a6d29825f Mon Sep 17 00:00:00 2001 From: marxin Date: Thu, 22 Sep 2016 11:18:49 +0200 Subject: [PATCH] Fix PR ipa/77653 gcc/testsuite/ChangeLog: 2016-09-22 Martin Liska PR ipa/77653 * gcc.dg/ipa/pr77653.c: New test. gcc/ChangeLog: 2016-09-22 Martin Liska PR ipa/77