Re: [PATCH] Fix PR ipa/68035

2015-11-13 Thread Martin Liška
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

Re: [PATCH] Fix PR ipa/68035 (v2)

2015-11-12 Thread Martin Liška
tests and bootstraps on x86_64-linux-pc. Ready for trunk? Thanks, Martin >From 29be4ad798d73245715f53fe971a17664b69eeb8 Mon Sep 17 00:00:00 2001 From: marxin <mli...@suse.cz> Date: Thu, 5 Nov 2015 18:31:31 +0100 Subject: [PATCH] Fix PR ipa/68035 gcc/ChangeLog: 2015-11-12 Martin Li

Re: [PATCH] Fix PR ipa/68035

2015-11-12 Thread Martin Liška
pc. >> >> Ready for trunk? >> Thanks, >> Martin > >> >From 62266e21a89777c6dbd680f7c87f15abe603c024 Mon Sep 17 00:00:00 2001 >> From: marxin <mli...@suse.cz> >> Date: Thu, 5 Nov 2015 18:31:31 +0100 >> Subject: [PATCH] Fix PR ipa/68035 >>

Re: [PATCH] Fix PR ipa/68035 (v2)

2015-11-12 Thread Richard Biener
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

Re: [PATCH] Fix PR ipa/68035

2015-11-12 Thread Jan Hubicka
> >> + > >> + /* 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

[PATCH] Fix PR ipa/68035

2015-11-06 Thread Martin Liška
:00 2001 From: marxin <mli...@suse.cz> Date: Thu, 5 Nov 2015 18:31:31 +0100 Subject: [PATCH] Fix PR ipa/68035 gcc/testsuite/ChangeLog: 2015-11-05 Martin Liska <mli...@suse.cz> * gcc.dg/ipa/pr68035.c: New test. gcc/ChangeLog: 2015-11-05 Martin Liska <mli...@suse.cz> PR i

Re: [PATCH] Fix PR ipa/68035

2015-11-06 Thread Richard Biener
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

Re: [PATCH] Fix PR ipa/68035

2015-11-06 Thread Jan Hubicka
; Thanks, > Martin > >From 62266e21a89777c6dbd680f7c87f15abe603c024 Mon Sep 17 00:00:00 2001 > From: marxin <mli...@suse.cz> > Date: Thu, 5 Nov 2015 18:31:31 +0100 > Subject: [PATCH] Fix PR ipa/68035 > > gcc/testsuite/ChangeLog: > > 2015-11-05 Martin Liska <mli...@suse.cz> > >