Re: [PATCH] Locality cloning pass

2025-04-16 Thread Richard Biener
On Wed, Apr 16, 2025 at 11:08 AM xionghuluo wrote: > > Hi, the bootstrap-lto-locality is much longer compared to boostrap-lto > and bootstrap, and > > It seems that stage2 and stage3 only produced 5 partitions in LTO, is > this reasonable... Likely due to the high default of -param=lto-max-local

Re: [PATCH] Locality cloning pass

2025-04-16 Thread xionghuluo
Hi, the bootstrap-lto-locality is much longer compared to boostrap-lto and bootstrap, and It seems that stage2 and stage3 only produced 5 partitions in LTO, is this reasonable... Also could you please inform how much is the exact performance gain, please? make bootstrap:

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-15 Thread Richard Biener
On Mon, Apr 14, 2025 at 3:11 PM Kyrylo Tkachov wrote: > > Hi Honza, > > > On 13 Apr 2025, at 23:19, Jan Hubicka wrote: > > > >> +@opindex fipa-reorder-for-locality > >> +@item -fipa-reorder-for-locality > >> +Group call chains close together in the binary layout to improve code code > >> +localit

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-15 Thread Kyrylo Tkachov
> On 15 Apr 2025, at 15:42, Richard Biener wrote: > > On Mon, Apr 14, 2025 at 3:11 PM Kyrylo Tkachov wrote: >> >> Hi Honza, >> >>> On 13 Apr 2025, at 23:19, Jan Hubicka wrote: >>> +@opindex fipa-reorder-for-locality +@item -fipa-reorder-for-locality +Group call chains close

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-14 Thread Kyrylo Tkachov
Hi Honza, > On 13 Apr 2025, at 23:19, Jan Hubicka wrote: > >> +@opindex fipa-reorder-for-locality >> +@item -fipa-reorder-for-locality >> +Group call chains close together in the binary layout to improve code code >> +locality. This option is incompatible with an explicit >> +@option{-flto-part

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-13 Thread Jan Hubicka
> +@opindex fipa-reorder-for-locality > +@item -fipa-reorder-for-locality > +Group call chains close together in the binary layout to improve code code > +locality. This option is incompatible with an explicit > +@option{-flto-partition=} option since it enforces a custom partitioning > +scheme.

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-10 Thread Kyrylo Tkachov
> On 26 Mar 2025, at 08:42, Kyrylo Tkachov wrote: > > Ping. Ping. https://gcc.gnu.org/pipermail/gcc-patches/2025-March/676958.html I’ve ran a profiled LTO bootstrap of GCC with the new bootstrap-lto-locality bootstrap config And compared it against a GCC produced by the existing lto-bootstra

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-03-26 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill > On 6 Mar 2025, at 09:25, Kyrylo Tkachov wrote: > > Hi all, > > Implement partitioning and cloning in the callgraph to help locality. > A new -fipa-reorder-for-locality flag is used to enable this. > The majority of the logic is in the new IPA pass in ipa-locality-cloning