Re: Merge IPA and late local modref flags

2021-11-09 Thread Marek Polacek via Gcc-patches
On Tue, Nov 09, 2021 at 05:32:42PM +0100, Jan Hubicka via Gcc-patches wrote: > > > + } > > > + if (!(flags & EAF_UNUSED)) > > > + lags |= past; > >    ^ > > > > > > Broke bootstrap. > Martin just fixed it. Sorry for that. > Diff complained about 8 spaces instead of tab an

Re: Merge IPA and late local modref flags

2021-11-09 Thread Jan Hubicka via Gcc-patches
> > + } > > + if (!(flags & EAF_UNUSED)) > > + lags |= past; >    ^ > > > Broke bootstrap. Martin just fixed it. Sorry for that. Diff complained about 8 spaces instead of tab and I did not rebuild after replacing it bit too overzelaously. Honza > > jeff >

Re: Merge IPA and late local modref flags

2021-11-09 Thread Jeff Law via Gcc-patches
On 11/9/2021 9:13 AM, Jan Hubicka via Gcc-patches wrote: Hi, since at the time we compute local solution during late modref the summaries from IPA are readily available (and I added logic to compare them), it is easy to intersect both solutions to get around cases where late optimization obstr

Merge IPA and late local modref flags

2021-11-09 Thread Jan Hubicka via Gcc-patches
Hi, since at the time we compute local solution during late modref the summaries from IPA are readily available (and I added logic to compare them), it is easy to intersect both solutions to get around cases where late optimization obstructate code enough so flags are no longer analyzed correctly.