Re: [PATCH v2] Rearrange SLP nodes with duplicate statements. [PR98138]

2024-08-08 Thread Richard Biener
On Tue, Aug 6, 2024 at 12:38 PM Manolis Tsamis wrote: > > Pinging this for a review and/or further feedback. > > Thanks, > Manolis > > On Wed, Jun 26, 2024 at 3:06 PM Manolis Tsamis > wrote: > > > > This change checks when a two_operators SLP node has multiple occurrences of > > the same stateme

Re: [PATCH v2] Rearrange SLP nodes with duplicate statements. [PR98138]

2024-08-06 Thread Manolis Tsamis
Pinging this for a review and/or further feedback. Thanks, Manolis On Wed, Jun 26, 2024 at 3:06 PM Manolis Tsamis wrote: > > This change checks when a two_operators SLP node has multiple occurrences of > the same statement (e.g. {A, B, A, B, ...}) and tries to rearrange the > operands > so that

Re: [PATCH v2] Rearrange SLP nodes with duplicate statements. [PR98138]

2024-06-26 Thread Manolis Tsamis
This is a reworked implementation that only deduplicates two_operators nodes and supports arbitrary orderings. Based on the discussion on the original submission, I have made some SPEC runs to see if this transformation applies to any other benchmarks. Aside from x264 this now triggers on SPEC2017