On Mon, 20 Jun 2011, Richard Guenther wrote:
> On Sun, Jun 19, 2011 at 2:51 PM, Jan Hubicka <hubi...@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 > > >
