From patchwork Thu Sep 20 22:14:39 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [ARM,testsuite] improve test for builtin_bswap16 for Thumb2
Date: Thu, 20 Sep 2012 12:14:39 -0000
From: Christophe Lyon <christophe.lyon@linaro.org>
X-Patchwork-Id: 185516
Message-Id: <CAKdteOZrqZ-f_7HLMOxULZqTRs56bmmroScfctdWKQuBR+bq6g@mail.gmail.com>
To: gcc-patches@gcc.gnu.org

Hi,

The attached patch make sure we test builtin_bswap16 expansion in
Thumb2 mode in addition to the existing Thumb1 test (which may fail as
unsupported if the compiler is also using float-abi=hard).

The newly created test is the same as the existing one, except for the
forced architecture version.

Christophe.
2012-09-21  Christophe Lyon <christophe.lyon@linaro.org>

	gcc/testsuite:
	* gcc.target/arm/builtin-bswap-1.c: Add comment.
	* gcc.target/arm/builtin-bswap-v6t2.c: New test.


diff --git a/gcc/testsuite/gcc.target/arm/builtin-bswap-1.c b/gcc/testsuite/gcc.target/arm/builtin-bswap-1.c
index 43195bd..fe95fb1 100644
--- a/gcc/testsuite/gcc.target/arm/builtin-bswap-1.c
+++ b/gcc/testsuite/gcc.target/arm/builtin-bswap-1.c
@@ -1,3 +1,4 @@
+/* Force testing on armv6 to check thumb1 mode.  */
 /* { dg-do compile } */
 /* { dg-options "-O2" } */
 /* { dg-require-effective-target arm_arch_v6_ok } */
diff --git a/gcc/testsuite/gcc.target/arm/builtin-bswap-v6t2.c b/gcc/testsuite/gcc.target/arm/builtin-bswap-v6t2.c
new file mode 100644
index 0000000..4ab3f85
--- /dev/null
+++ b/gcc/testsuite/gcc.target/arm/builtin-bswap-v6t2.c
@@ -0,0 +1,82 @@
+/* Force testing on armv6t2 to check thumb2 mode.  */
+/* { dg-do compile } */
+/* { dg-options "-O2" } */
+/* { dg-require-effective-target arm_arch_v6t2_ok } */
+/* { dg-add-options arm_arch_v6t2 } */
+/* { dg-final { scan-assembler-not "orr\[ \t\]" } } */
+/* { dg-final { scan-assembler-times "revsh\\t" 1 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "revshne\\t" 1 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "revsh\\t" 2 { target { ! arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "rev16\\t" 1 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "rev16ne\\t" 1 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "rev16\\t" 2 { target { ! arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "rev\\t" 2 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "revne\\t" 2 { target { arm_nothumb } } } }  */
+/* { dg-final { scan-assembler-times "rev\\t" 4 { target { ! arm_nothumb } } } }  */
+
+/* revsh */
+short swaps16 (short x)
+{
+  return __builtin_bswap16 (x);
+}
+
+extern short foos16 (short);
+
+/* revshne */
+short swaps16_cond (short x, int y)
+{
+  short z = x;
+  if (y)
+    z = __builtin_bswap16 (x);
+  return foos16 (z);
+}
+
+/* rev16 */
+unsigned short swapu16 (unsigned short x)
+{
+  return __builtin_bswap16 (x);
+}
+
+extern unsigned short foou16 (unsigned short);
+
+/* rev16ne */
+unsigned short swapu16_cond (unsigned short x, int y)
+{
+  unsigned short z = x;
+  if (y)
+    z = __builtin_bswap16 (x);
+  return foou16 (z);
+}
+
+/* rev */
+int swaps32 (int x) {
+  return __builtin_bswap32 (x);
+}
+
+extern int foos32 (int);
+
+/* revne */
+int swaps32_cond (int x, int y)
+{
+  int z = x;
+  if (y)
+    z = __builtin_bswap32 (x);
+  return foos32 (z);
+}
+
+/* rev */
+unsigned int swapu32 (unsigned int x)
+{
+  return __builtin_bswap32 (x);
+}
+
+extern unsigned int foou32 (unsigned int);
+
+/* revne */
+unsigned int swapsu2 (unsigned int x, int y)
+{
+  int z = x;
+  if (y)
+    z = __builtin_bswap32 (x);
+  return foou32 (z);
+}