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

Hans-Peter Nilsson hp@bitrange.com
Mon Jun 20 14:47:00 GMT 2011



On Mon, 20 Jun 2011, Richard Guenther wrote:

> On Sun, Jun 19, 2011 at 2:51 PM, Jan Hubicka <hubicka@ucw.cz> 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
> >> > > function nodes that have IPA_REF_ALIAS reference to the node they are alias of.
> >> > >
> >> > > I am still getting one failure:
> >> > > FAIL: g++.dg/torture/pr43879-1_1.C
> >> > >
> >> > > It tests IPA PTA in presence of same body aliases.  I honestly have no idea
> >> > > what is wrong there.  I decided to go ahead with the patch anyway, given the
> >> > > current state of affair of aliases and IPA-PTA. Hope Richard will help me
> >> > > fixing this on Monday.
> >> >
> >> > Still there, see again PR49373.  Anything better than a xfail in the works?
> >>
> >> Yes, we discussed the problem with Richi and the issue is that ipa-pta gets
> >> confused by inliner redirecting edge from alias to the real inline clone.
> >> The proper fix is to teach passmanager to do small ipa passes before final
> >> compilation.  I am currently at the GCC gathering, so I plan to implement
> >> this day after tomorrow after returning.
> > But for sure we can xfail it.
> > (It was prevoiusly latent problem and richi did quicker fix but that just
> > uncovered another problems. Sorry for the delays here).
>
> If you xfail it please open a regression bugreport.

Anything wrong with the already open PR49373 where it's
mentioned?

>
> Richard.
>
> > Honza
> >
>


More information about the Gcc-patches mailing list