Re: [PATCH 2/2] PowerPC: update comments for the MMA instruction name changes.

[Carl Love <cel@us.ibm.com>]

Ulrich:

On Fri, 2022-11-04 at 13:04 +0000, Ulrich Weigand wrote:
> Carl Love <cel@us.ibm.com> wrote:
> 
> > PowerPC update comments for the MMA instruction name changes.
> 
> The *comment* changes are OK.  However, this:
> 
> -  __asm__ __volatile__ ("pmxvi8ger4spp  6, %x0, %x1, 11, 13, 5"
> +  __asm__ __volatile__ ("pmdmxvi8ger4spp  6, %x0, %x1, 11, 13, 5"
>                                  :: "wa" (vec_xa), "wa" (vec_xb) );
> -  __asm__ __volatile__ ("pmxvf32gerpp  7, %x0, %x1, 11, 13"
> +  __asm__ __volatile__ ("pmdmxvf32gerpp  7, %x0, %x1, 11, 13"
>                                  :: "wa" (vec_xa), "wa" (vec_xb) );
> 
> actually changes code, and in particular it requires that the
> compiler and/or assembler used to build the test case understands
> the new name.  This would cause the test to fail when run on a
> system with an older toolchain.
> 
> I think this part of the patch should be removed.

Yup, forgot about that change.  I have reverted it back to the older
names and put a comment in stating that the test uses the older names
for backward compatibility.  Thanks.

I will post version 2 of the patch.

                     Carl