Hi Avinash, While the below changes look good, there are a few more fixes to be made to the AES builtins patch:
1. In the extend.texi file, please add a subheading stating that the content is for AES builtins. For reference, you can look at the ECC patch. You will have to add @subsubheading. 2. In the rs6000-builtins.def file, the builtins have to be kept in future-vsx stanza. You can resend this patch with the changes. Please change the subject line. -Surya On 01/06/26 12:24 PM, Avinash Jayakar wrote: > From: Avinash Jayakar <[email protected]> > > This patch depends on > https://gcc.gnu.org/pipermail/gcc-patches/2026-May/718172.html > > Effective target directives were missed out when adding AES builtins > for powerpc, this patch adds them for the affected tests. > > 2026-06-01 Avinash Jayakar <[email protected]> > > gcc/testsuite/ChangeLog: > * gcc.target/powerpc/aes-builtin-1.c: Add future_compile_ok and lp64 > testsuite directive. > * gcc.target/powerpc/aes-builtin-2.c: Add future_compile_ok and lp64 > testsuite directive. > * gcc.target/powerpc/aes-builtin-3.c: Add future_compile_ok and lp64 > testsuite directive. > --- > gcc/testsuite/gcc.target/powerpc/aes-builtin-1.c | 2 ++ > gcc/testsuite/gcc.target/powerpc/aes-builtin-2.c | 2 ++ > gcc/testsuite/gcc.target/powerpc/aes-builtin-3.c | 2 ++ > 3 files changed, 6 insertions(+) > > diff --git a/gcc/testsuite/gcc.target/powerpc/aes-builtin-1.c > b/gcc/testsuite/gcc.target/powerpc/aes-builtin-1.c > index 2777fcdc7ae..729ae1ef26d 100644 > --- a/gcc/testsuite/gcc.target/powerpc/aes-builtin-1.c > +++ b/gcc/testsuite/gcc.target/powerpc/aes-builtin-1.c > @@ -1,4 +1,6 @@ > /* { dg-do compile } */ > +/* { dg-require-effective-target powerpc_future_compile_ok } */ > +/* { dg-require-effective-target lp64 } */ > /* { dg-options "-mdejagnu-cpu=future -O2" } */ > > void > diff --git a/gcc/testsuite/gcc.target/powerpc/aes-builtin-2.c > b/gcc/testsuite/gcc.target/powerpc/aes-builtin-2.c > index 98f16ad0fe8..a509f760cd6 100644 > --- a/gcc/testsuite/gcc.target/powerpc/aes-builtin-2.c > +++ b/gcc/testsuite/gcc.target/powerpc/aes-builtin-2.c > @@ -1,4 +1,6 @@ > /* { dg-do compile } */ > +/* { dg-require-effective-target powerpc_future_compile_ok } */ > +/* { dg-require-effective-target lp64 } */ > /* { dg-options "-mdejagnu-cpu=future -O2" } */ > > typedef unsigned char vec_t __attribute__((vector_size(16))); > diff --git a/gcc/testsuite/gcc.target/powerpc/aes-builtin-3.c > b/gcc/testsuite/gcc.target/powerpc/aes-builtin-3.c > index 007bf878afb..bbed43a1e8c 100644 > --- a/gcc/testsuite/gcc.target/powerpc/aes-builtin-3.c > +++ b/gcc/testsuite/gcc.target/powerpc/aes-builtin-3.c > @@ -1,4 +1,6 @@ > /* { dg-do compile } */ > +/* { dg-require-effective-target powerpc_future_compile_ok } */ > +/* { dg-require-effective-target lp64 } */ > /* { dg-options "-mdejagnu-cpu=future -O2" } */ > > void
