Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-22 Thread Hans-Peter Nilsson
On Sun, 19 Jun 2011, Jan Hubicka wrote: > > > On Sat, 11 Jun 2011, Jan Hubicka wrote: > > > > > > > Hi, > > > > this patch complettes the same body alias rework by removing the old > > > > same body > > > > alias code and adding new representation. Same body aliases are now > > > > separate > >

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-20 Thread Hans-Peter Nilsson
On Mon, 20 Jun 2011, Richard Guenther wrote: > On Sun, Jun 19, 2011 at 2:51 PM, Jan Hubicka wrote: > >> > On Sat, 11 Jun 2011, Jan Hubicka wrote: > >> > > >> > > Hi, > >> > > this patch complettes the same body alias rework by removing the old > >> > > same body > >> > > alias code and adding

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-20 Thread Richard Guenther
On Sun, Jun 19, 2011 at 2:51 PM, Jan Hubicka wrote: >> > On Sat, 11 Jun 2011, Jan Hubicka wrote: >> > >> > > Hi, >> > > this patch complettes the same body alias rework by removing the old >> > > same body >> > > alias code and adding new representation.  Same body aliases are now >> > > separat

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-19 Thread Jan Hubicka
> > On Sat, 11 Jun 2011, Jan Hubicka wrote: > > > > > Hi, > > > this patch complettes the same body alias rework by removing the old same > > > body > > > alias code and adding new representation. Same body aliases are now > > > separate > > > function nodes that have IPA_REF_ALIAS reference to

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-19 Thread Jan Hubicka
> On Sat, 11 Jun 2011, Jan Hubicka wrote: > > > Hi, > > this patch complettes the same body alias rework by removing the old same > > body > > alias code and adding new representation. Same body aliases are now > > separate > > function nodes that have IPA_REF_ALIAS reference to the node they a

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-19 Thread Hans-Peter Nilsson
On Sat, 11 Jun 2011, Jan Hubicka wrote: > Hi, > this patch complettes the same body alias rework by removing the old same body > alias code and adding new representation. Same body aliases are now separate > function nodes that have IPA_REF_ALIAS reference to the node they are alias > of. > > I

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread Jan Hubicka
Hi, this patch solves the bultin/strlen-3.c LTO linker plugin problem. While removing alias code I was bit overactive and removed the check that makes us to implicitly do -fwhole-program when resolution info is around. It is not quite clear to me why in LTO we need -fwhole-program to get the tes

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread H.J. Lu
On Sun, Jun 12, 2011 at 9:40 AM, Jan Hubicka wrote: > Hi, > this is the fix (or rather a workaround) i comitted.  Thanks! > > Index: ChangeLog > === > --- ChangeLog   (revision 174968) > +++ ChangeLog   (working copy) > @@ -1,3 +1,9 @

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread H.J. Lu
On Sun, Jun 12, 2011 at 9:33 AM, Jan Hubicka wrote: >> On Sun, Jun 12, 2011 at 7:54 AM, Jan Hubicka wrote: >> >> >> >> This also pretty much destroyed C++ for ia32: >> >> >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 >> >> http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html >> >

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread Jan Hubicka
Hi, this is the fix (or rather a workaround) i comitted. Thanks! Index: ChangeLog === --- ChangeLog (revision 174968) +++ ChangeLog (working copy) @@ -1,3 +1,9 @@ +2011-06-11 Jan Hubicka + + PR middle-end/49378 + *

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread Jan Hubicka
> On Sun, Jun 12, 2011 at 7:54 AM, Jan Hubicka wrote: > >> > >> This also pretty much destroyed C++ for ia32: > >> > >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 > >> http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html > > > > Hi, > > It seems somewhat amazing that we hit kernel sen

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread H.J. Lu
On Sun, Jun 12, 2011 at 8:01 AM, H.J. Lu wrote: > On Sun, Jun 12, 2011 at 7:54 AM, Jan Hubicka wrote: >>> >>> This also pretty much destroyed C++ for ia32: >>> >>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 >>> http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html >> >> Hi, >> It seem

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread H.J. Lu
On Sun, Jun 12, 2011 at 7:54 AM, Jan Hubicka wrote: >> >> This also pretty much destroyed C++ for ia32: >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 >> http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html > > Hi, > It seems somewhat amazing that we hit kernel sensitive miscompilat

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread Jan Hubicka
> I am testing it now. Will know the results in 2 hours. Thanks. Could you also send me the preprocessed source for future.o? The object file I am getting don't have the bug you report, that is most probably due to glibc difference. Honza > > Thanks. > > -- > H.J.

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread H.J. Lu
On Sun, Jun 12, 2011 at 7:54 AM, Jan Hubicka wrote: >> >> This also pretty much destroyed C++ for ia32: >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 >> http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html > > Hi, > It seems somewhat amazing that we hit kernel sensitive miscompilat

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-12 Thread Jan Hubicka
> > This also pretty much destroyed C++ for ia32: > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49378 > http://gcc.gnu.org/ml/gcc-regression/2011-06/msg00159.html Hi, It seems somewhat amazing that we hit kernel sensitive miscompilation here. The problem most probably is the fact that thunks a

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-11 Thread H.J. Lu
On Sat, Jun 11, 2011 at 9:15 AM, H.J. Lu wrote: > On Sat, Jun 11, 2011 at 5:55 AM, Jan Hubicka wrote: >> Hi, >> this patch complettes the same body alias rework by removing the old same >> body >> alias code and adding new representation.  Same body aliases are now separate >> function nodes tha

Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases)

2011-06-11 Thread H.J. Lu
On Sat, Jun 11, 2011 at 5:55 AM, Jan Hubicka wrote: > Hi, > this patch complettes the same body alias rework by removing the old same body > alias code and adding new representation.  Same body aliases are now separate > function nodes that have IPA_REF_ALIAS reference to the node they are alias