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

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

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

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) {

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