Re: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Jan Hubicka
> > > BTW: I have never seen any VAR_DECL here, at least not in my test case, do > you have test cases for that? If you call virtual method of undefined variable: struct A { virtual void foo (); }; void t() { struct A *a; a->foo (); } It will make GCC to drop the virtual table lookup and

RE: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Bernd Edlinger
Hi Honza, On Tue, 9 Dec 2014 00:16:36, Jan Hubicka wrote: > >>> On Mon, Dec 8, 2014 at 11:57 AM, Bernd Edlinger >>> wrote: > > Hello, > > > this surprisingly simple patch fixes PR ipa/64049. The root cause seems > to be > that in this test case we try to de

Re: FW: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Jan Hubicka
> > On Mon, Dec 8, 2014 at 11:57 AM, Bernd Edlinger > > wrote: > > > > > > > > >> > > >> Hello, > > >> > > >> > > >> this surprisingly simple patch fixes PR ipa/64049. The root cause seems > > >> to be > > >> that in this test case we try to devirtualize a method call on a return > > >> value.

Re: FW: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Jan Hubicka
> On Mon, Dec 8, 2014 at 11:57 AM, Bernd Edlinger > wrote: > > > > > >> > >> Hello, > >> > >> > >> this surprisingly simple patch fixes PR ipa/64049. The root cause seems > >> to be > >> that in this test case we try to devirtualize a method call on a return > >> value. > >> > >> > >> Boot-Stra

Re: FW: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Richard Biener
On Mon, Dec 8, 2014 at 11:57 AM, Bernd Edlinger wrote: > > >> >> Hello, >> >> >> this surprisingly simple patch fixes PR ipa/64049. The root cause seems to >> be >> that in this test case we try to devirtualize a method call on a return >> value. >> >> >> Boot-Strapped and regression-tested on

FW: [PATCH] Fix PR ipa/64049

2014-12-08 Thread Bernd Edlinger
> > Hello, > > > this surprisingly simple patch fixes PR ipa/64049. The root cause seems to be > that in this test case we try to devirtualize a method call on a return value. > > > Boot-Strapped and regression-tested on X86_64-linux-gnu. > OK for trunk? > > > Thanks > Bernd. > Again with chang

[PATCH] Fix PR ipa/64049

2014-12-08 Thread Bernd Edlinger
Hello, this surprisingly simple patch fixes PR ipa/64049.  The root cause seems to be that in this test case we try to devirtualize a method call on a return value. Boot-Strapped and regression-tested on X86_64-linux-gnu. OK for trunk? Thanks Bernd.