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