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.
TIA
