On 11/12/2015 07:40 PM, Jan Hubicka wrote:
+
+ /* Initialize hash values if we are not in LTO mode. */
+ if (!in_lto_p)
+ item->get_hash ();
}
>>>
>>> Hmm, what is the difference to the LTO mode here. I would have expected
>>> that all the items
>>> was an
> >> +
> >> + /* Initialize hash values if we are not in LTO mode. */
> >> + if (!in_lto_p)
> >> + item->get_hash ();
> >> }
> >
> > Hmm, what is the difference to the LTO mode here. I would have expected
> > that all the items
> > was analyzed in both paths?
>
> Difference is t
On Thu, Nov 12, 2015 at 10:48 AM, Martin Liška wrote:
> Hello.
>
> I'm sending reworked version of the patch, where I renamed 'sem_item::hash'
> to 'm_hash'
> and wrapped all usages with 'get_hash'. Apart from that, a new member
> function 'set_hash'
> is utilized for changing the hash value. Ho
pc.
>>
>> Ready for trunk?
>> Thanks,
>> Martin
>
>> >From 62266e21a89777c6dbd680f7c87f15abe603c024 Mon Sep 17 00:00:00 2001
>> From: marxin
>> Date: Thu, 5 Nov 2015 18:31:31 +0100
>> Subject: [PATCH] Fix PR ipa/68035
>>
>> gcc/testsu
r for understanding.
Patch can survive regression tests and bootstraps on x86_64-linux-pc.
Ready for trunk?
Thanks,
Martin
>From 29be4ad798d73245715f53fe971a17664b69eeb8 Mon Sep 17 00:00:00 2001
From: marxin
Date: Thu, 5 Nov 2015 18:31:31 +0100
Subject: [PATCH] Fix PR ipa/68035
gcc/ChangeLog:
2
; Thanks,
> Martin
> >From 62266e21a89777c6dbd680f7c87f15abe603c024 Mon Sep 17 00:00:00 2001
> From: marxin
> Date: Thu, 5 Nov 2015 18:31:31 +0100
> Subject: [PATCH] Fix PR ipa/68035
>
> gcc/testsuite/ChangeLog:
>
> 2015-11-05 Martin Liska
>
> * gcc.dg/ipa/pr68035.c: New test.
>
On Fri, Nov 6, 2015 at 10:10 AM, Martin Liška wrote:
> Hello.
>
> Following patch triggers hash calculation of items (functions and variables)
> in situations where LTO mode is not utilized.
>
> Patch survives regression tests and bootstraps on x86_64-linux-pc.
Why does that make a difference? D
:00 2001
From: marxin
Date: Thu, 5 Nov 2015 18:31:31 +0100
Subject: [PATCH] Fix PR ipa/68035
gcc/testsuite/ChangeLog:
2015-11-05 Martin Liska
* gcc.dg/ipa/pr68035.c: New test.
gcc/ChangeLog:
2015-11-05 Martin Liska
PR ipa/68035
* ipa-icf.c (sem_item_optimizer::build_graph): Fo