[PATCH v2] rs6000: Support doubleword swaps removal in rot64 load store [PR100085]

Segher Boessenkool segher@kernel.crashing.org
Wed Jun 9 16:47:37 GMT 2021


On Wed, Jun 09, 2021 at 11:06:31AM +0800, Xionghu Luo wrote:
> On 2021/6/9 05:07, Segher Boessenkool wrote:
> >> -/* { dg-final { scan-assembler "lxvd2x 34"  } } */
> >> -/* { dg-final { scan-assembler "stxvd2x 34" } } */
> >> +/* { dg-final { scan-assembler "lvx 2"  } } */
> >> +/* { dg-final { scan-assembler "stvx 2" } } */
> > 
> > Huh.  Is that correct?  Where did the other 32 loads and stores go?  Are
> > there now other insns generated that you should scan for?
> 
> This is expected change. lxvd2x+xxpermdi is replaced by lvx.  No need scan other
> instructions. Similarly for stvx. 34 and 2 are *vector register names* instead of
> counts.

Oh!  Oh my.  I read that as "scan-assembler-times" for some reason, as
you expected.

> Thanks for all the other comments, updated and committed with r12-1316.

Thank you!


Segher


More information about the Gcc-patches mailing list