Hi Peter,

On Wed, Jul 22, 2020 at 12:01:21PM -0500, Peter Bergner wrote:
> PR96236 shows a problem where we don't correctly store our 512-bit 
> accumulators
> correctly in little-endian mode.  The patch below detects when we're doing a
> little-endian memory access and stores to the correct memory locations.

> gcc/
>       PR target/96236
>       * config/rs6000/rs6000-call.c (rs6000_gimple_fold_mma_builtin): Handle
>       little-endian memory ordering.
>     
> gcc/testsuite/
>       PR target/96236
>       * gcc.target/powerpc/mma-double-test.c: Update storing results for
>       correct little-endian ordering.
>       * gcc.target/powerpc/mma-single-test.c: Likewise.

Okay for trunk.  It's not going to benefit from any soak-in time other
than what you have tested already, so it is fine for 10 immediately as
well.  Thanks!


Segher

Reply via email to