public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [GCC] Add aarch64 to zero_bits_compound testing
@ 2016-06-10 13:57 Andre Vieira (lists)
  2016-06-14  9:54 ` James Greenhalgh
  0 siblings, 1 reply; 2+ messages in thread
From: Andre Vieira (lists) @ 2016-06-10 13:57 UTC (permalink / raw)
  To: GCC Patches; +Cc: vogt, law

[-- Attachment #1: Type: text/plain, Size: 394 bytes --]

Hello,

This patch adds aarch64*-*-* to the list of supported targets for the
recently added zero bits compound tests.
Tested for aarch64-none-elf and aarch64_be-none-elf.

Is this OK?

Cheers,
Andre

gcc/testsuite/ChangeLog
2016-06-10  Andre Vieira  <andre.simoesdiasvieira@arm.com>

        * gcc.dg/zero_bits_compound-1.c: Support aarch64.
        * gcc.dg/zero_bits_compound-1.c: Likewise.

[-- Attachment #2: zero_bits_compound.patch --]
[-- Type: text/x-patch, Size: 1327 bytes --]

diff --git a/gcc/testsuite/gcc.dg/zero_bits_compound-1.c b/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
index d78dc43d0a4d9c22b9c19a8435ca0f976b9819b6..650da60c0c33f912fd94fa330551a809d1d0fe67 100644
--- a/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
+++ b/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
@@ -4,7 +4,7 @@
 /* Note: This test requires that char, int and long have different sizes and the
    target has a way to do 32 -> 64 bit zero extension other than AND.  */
 
-/* { dg-do compile { target x86_64-*-* s390*-*-* } } */
+/* { dg-do compile { target x86_64-*-* s390*-*-* aarch64*-*-* } } */
 /* { dg-require-effective-target lp64 } */
 /* { dg-options "-O3 -dP" } */
 
diff --git a/gcc/testsuite/gcc.dg/zero_bits_compound-2.c b/gcc/testsuite/gcc.dg/zero_bits_compound-2.c
index 80fd363d9552e221d48801d2f29717ca5f3a42d4..f282b94d77915fd1717f3a51dc35c12682453f85 100644
--- a/gcc/testsuite/gcc.dg/zero_bits_compound-2.c
+++ b/gcc/testsuite/gcc.dg/zero_bits_compound-2.c
@@ -1,7 +1,7 @@
 /* Test whether an AND mask or'ed with the know zero bits that equals a mode
    mask is a candidate for zero extendion.  */
 
-/* { dg-do compile { target x86_64-*-* s390*-*-* } } */
+/* { dg-do compile { target x86_64-*-* s390*-*-* aarch64*-*-* } } */
 /* { dg-require-effective-target lp64 } */
 /* { dg-options "-O3 -dP" } */
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GCC] Add aarch64 to zero_bits_compound testing
  2016-06-10 13:57 [GCC] Add aarch64 to zero_bits_compound testing Andre Vieira (lists)
@ 2016-06-14  9:54 ` James Greenhalgh
  0 siblings, 0 replies; 2+ messages in thread
From: James Greenhalgh @ 2016-06-14  9:54 UTC (permalink / raw)
  To: Andre Vieira (lists); +Cc: GCC Patches, vogt, law, nd

On Fri, Jun 10, 2016 at 02:57:23PM +0100, Andre Vieira (lists) wrote:
> Hello,
> 
> This patch adds aarch64*-*-* to the list of supported targets for the
> recently added zero bits compound tests.
> Tested for aarch64-none-elf and aarch64_be-none-elf.
> 
> Is this OK?

This is OK, thanks. The tests already check for lp64, so I see no reason not
to enable them.

Thanks,
James

> gcc/testsuite/ChangeLog
> 2016-06-10  Andre Vieira  <andre.simoesdiasvieira@arm.com>
> 
>         * gcc.dg/zero_bits_compound-1.c: Support aarch64.
>         * gcc.dg/zero_bits_compound-1.c: Likewise.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-06-14  9:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-10 13:57 [GCC] Add aarch64 to zero_bits_compound testing Andre Vieira (lists)
2016-06-14  9:54 ` James Greenhalgh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).