Re: aliasing_component_refs_p tweek

2019-05-07 Thread Jan Hubicka
> > No need for the else {} and thus indenting the rest since the if () > arm always returns from the function. > > OK with eliding this else { } wrapping. Ah, right, I changed the function bit too many times :) Here is updated patch I re-tested and comitted. * tree-ssa-alias.c (aliasin

Re: aliasing_component_refs_p tweek

2019-05-06 Thread Richard Biener
On Mon, 6 May 2019, Jan Hubicka wrote: > Hi, > this patch makes aliasing_component_refs_p little bit stronger, especially > with LTO increasing number of disambiguations on tramp3d by 2%. > > It took me a while to uderstand that function so a bit summary what it > does. > > Function analyzes cha

aliasing_component_refs_p tweek

2019-05-06 Thread Jan Hubicka
Hi, this patch makes aliasing_component_refs_p little bit stronger, especially with LTO increasing number of disambiguations on tramp3d by 2%. It took me a while to uderstand that function so a bit summary what it does. Function analyzes chains of nested references like ref1: a.b.c.d.e ref2: c