Re: [PATCH, rs6000] Add store fusion support for Power10

2021-08-04 Thread Segher Boessenkool
Hi! On Wed, Aug 04, 2021 at 04:16:45PM -0500, Pat Haugen wrote: > On 8/4/21 9:23 AM, Bill Schmidt wrote: > >> +  /* GPR stores can be ascending or descending offsets, FPR/VSR stores > > VSR?  I don't see how that applies here. Almost all scalar FP insns have a VR alternative, but unfortunatel

Re: [PATCH, rs6000] Add store fusion support for Power10

2021-08-04 Thread Pat Haugen via Gcc-patches
On 8/4/21 9:23 AM, Bill Schmidt wrote: > Hi Pat, > > Good stuff!  Comments below. > > On 8/2/21 3:19 PM, Pat Haugen via Gcc-patches wrote: >> diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c >> index 279f00cc648..1460a0d7c5c 100644 >> --- a/gcc/config/rs6000/rs6000.c >> +++ b/

Re: [PATCH, rs6000] Add store fusion support for Power10

2021-08-04 Thread Segher Boessenkool
On Wed, Aug 04, 2021 at 09:23:13AM -0500, Bill Schmidt wrote: > On 8/2/21 3:19 PM, Pat Haugen via Gcc-patches wrote: (I reviewed this elsewhere instead of on the list... Not good, since the patch was on the list already. Sorry.) > >@@ -18885,6 +18980,10 @@ rs6000_sched_reorder (FILE *dump > >A

Re: [PATCH, rs6000] Add store fusion support for Power10

2021-08-04 Thread Bill Schmidt via Gcc-patches
Hi Pat, Good stuff!  Comments below. On 8/2/21 3:19 PM, Pat Haugen via Gcc-patches wrote: Enable store fusion on Power10. Use the SCHED_REORDER hook to implement Power10 specific ready list reordering. As of now, pairing stores for store fusion is the only function being performed. Bootstrap/

[PATCH, rs6000] Add store fusion support for Power10

2021-08-02 Thread Pat Haugen via Gcc-patches
Enable store fusion on Power10. Use the SCHED_REORDER hook to implement Power10 specific ready list reordering. As of now, pairing stores for store fusion is the only function being performed. Bootstrap/regtest on powerpc64le(Power10) with no new regressions. Ok for master? -Pat 2021-08-02 Pa