> Hi honza,
>
> On 20 May 2019 11:38:14 CEST, Richard Biener <rguent...@suse.de> wrote:
> >On Mon, 20 May 2019, Jan Hubicka wrote:
> >
> >> Hi,
> >> at the moment the alias stats outputs mostly 0 for res_may_alias_p
> >> (which is supposed to be main entry point to the alias oracle).
> >> I think this is because of code refactoring where we added more
> >variants
> >> of this function and we want to do stats around the main worker.
> >>
> >> Bootstrapped/regtested x86_64-linux, OK?
> >
> >OK.
>
> >> @@ -1569,15 +1650,9 @@ bool
> >> refs_may_alias_p (tree ref1, tree ref2, bool tbaa_p)
> >> {
> >> ao_ref r1, r2;
> >> - bool res;
> >> ao_ref_init (&r1, ref1);
> >> ao_ref_init (&r2, ref2);
> >> - res = refs_may_alias_p_1 (&r1, &r2, tbaa_p);
> >> - if (res)
> >> - ++alias_stats.refs_may_alias_p_may_alias;
> >> - else
> >> - ++alias_stats.refs_may_alias_p_no_alias;
> >> - return res;
> >> + return; refs_may_alias_p_1 (&r1, &r2, tbaa_p);
> >> }
>
> Maybe remove the dead call to refs_may_alias_p_1() after the return though,
> please.
This is OK in trunk. I wonder how that got into the email :)
Thanks for noticing though.
Honza
>
> TIA
