Re: [PATCH] Fix PR ipa/61190, updated

2014-11-25 Thread Jan Hubicka
> > Index: gcc/ipa-pure-const.c > > === > > --- gcc/ipa-pure-const.c (revision 215888) > > +++ gcc/ipa-pure-const.c (working copy) > > @@ -744,6 +744,8 @@ analyze_function (struct cgraph_node *fn, bool ipa > > { > > /* Thunk gets propa

RE: [PATCH] Fix PR ipa/61190, updated

2014-11-25 Thread Bernd Edlinger
Hi Honza, On Mon, 24 Nov 2014 16:57:42 +0100, Jan Hubicka wrote: > > +cgraph_node::call_for_symbol_thunks_and_aliases_1 (bool (*callback) > + (cgraph_node *, void *), > + void *data, > + bool include_overwritable, > + bool exclude_virtual_thunks) > > Instead of adding _1 variant into public API,

Re: [PING**4] [PATCH] Fix PR ipa/61190, 2nd edition�??

2014-11-24 Thread Jan Hubicka
+cgraph_node::call_for_symbol_thunks_and_aliases_1 (bool (*callback) +(cgraph_node *, void *), + void *data, + bool include_overwritable, +

RE: [PING**4] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-11-24 Thread Bernd Edlinger
. > From: bernd.edlin...@hotmail.de > To: gcc-patches@gcc.gnu.org > CC: richard.guent...@gmail.com; hubi...@ucw.cz > Subject: RE: [PING**3] [PATCH] Fix PR ipa/61190, 2nd edition‏ > Date: Wed, 12 Nov 2014 13:43:18 +0100 > > > > Ping... > > > -

RE: [PING**3] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-11-12 Thread Bernd Edlinger
Ping... > From: bernd.edlin...@hotmail.de > To: gcc-patches@gcc.gnu.org > CC: richard.guent...@gmail.com; hubi...@ucw.cz > Subject: RE: [PING**2] [PATCH] Fix PR ipa/61190, 2nd edition‏ > Date: Mon, 27 Oct 2014 09:23:41 +0100 &

RE: [PING**2] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-10-27 Thread Bernd Edlinger
Ping again... Thanks. > From: bernd.edlin...@hotmail.de > To: hubi...@ucw.cz > CC: gcc-patches@gcc.gnu.org; richard.guent...@gmail.com > Subject: [PING] [PATCH] Fix PR ipa/61190, 2nd edition‏ > Date: Tue, 14 Oct 2014 11:40:56 +0200 >

[PING] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-10-14 Thread Bernd Edlinger
Ping... see: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00536.html > Hi Honza, > > > as you know, we have a wrong code bug, when a pure or const method is called > via a virtual thunk. > I had some more Ideas, how to fix that, but all of them had some serious > draw-backs, so I leave the det

[PATCH] Fix PR ipa/61190, 2nd edition‏

2014-10-07 Thread Bernd Edlinger
Hi Honza, as you know, we have a wrong code bug, when a pure or const method is called via a virtual thunk. I had some more Ideas, how to fix that, but all of them had some serious draw-backs, so I leave the details out... But now I have a new insight, why the "obvious" fix for this serious c

RE: [PATCH] Fix PR ipa/61190

2014-06-16 Thread Bernd Edlinger
Hi Honza, On Mon, 2 Jun 2014 18:12:10, Jan Hubicka wrote: > >> Hi, >> >> On Mon, 2 Jun 2014 12:06:12, Richard Biener wrote: >>> >>> On Mon, Jun 2, 2014 at 11:00 AM, Bernd Edlinger >>> wrote: Hi, the test case g++.old-deja/g++.mike/p4736b.C is mis-compiled with with all optimiz

Re: [PATCH] Fix PR ipa/61190

2014-06-02 Thread Jan Hubicka
> Hi, > > On Mon, 2 Jun 2014 12:06:12, Richard Biener wrote: > > > > On Mon, Jun 2, 2014 at 11:00 AM, Bernd Edlinger > > wrote: > >> Hi, > >> > >> the test case g++.old-deja/g++.mike/p4736b.C is mis-compiled with with all > >> optimization levels, except -O0 and -Og. This probably started with gc

RE: [PATCH] Fix PR ipa/61190

2014-06-02 Thread Bernd Edlinger
Hi, On Mon, 2 Jun 2014 12:06:12, Richard Biener wrote: > > On Mon, Jun 2, 2014 at 11:00 AM, Bernd Edlinger > wrote: >> Hi, >> >> the test case g++.old-deja/g++.mike/p4736b.C is mis-compiled with with all >> optimization levels, except -O0 and -Og. This probably started with gcc >> 4.7.x. >> >> T

Re: [PATCH] Fix PR ipa/61190

2014-06-02 Thread Richard Biener
On Mon, Jun 2, 2014 at 11:00 AM, Bernd Edlinger wrote: > Hi, > > the test case g++.old-deja/g++.mike/p4736b.C is mis-compiled with with all > optimization levels, except -O0 and -Og. This probably started with gcc 4.7.x. > > The constructor Main::Main() is first inlined and then completely optimiz

[PATCH] Fix PR ipa/61190

2014-06-02 Thread Bernd Edlinger
Hi, the test case g++.old-deja/g++.mike/p4736b.C is mis-compiled with with all optimization levels, except -O0 and -Og. This probably started with gcc 4.7.x. The constructor Main::Main() is first inlined and then completely optimized away in the dce1 pass. But the thunk is still using the vtable,