[PATCH, rs6000] Add store fusion support for Power10

Segher Boessenkool segher@kernel.crashing.org
Wed Aug 4 21:55:35 GMT 2021


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 unfortunately not
(always) with a unified or consistent syntax.

> > Can you think of any test cases we can use to demonstrate store fusion?
> 
> Yeah, should be able to come up with something to verify two adjacent stores.

The testcase should also stop working when the fusion does.  This might
mean you need more than one testcase.  The number one goal of our
testsuite is to warn of regressions, so it really should be more than
just a single two-liner scanning for one assembler insn.  Maybe do a
handful of cases?  :-)


Segher


More information about the Gcc-patches mailing list