Patchwork [ARM,testsuite] improve test for builtin_bswap16 for Thumb2

login
register
mail settings
Submitter Christophe Lyon
Date Sept. 20, 2012, 10:14 p.m.
Message ID <CAKdteOZrqZ-f_7HLMOxULZqTRs56bmmroScfctdWKQuBR+bq6g@mail.gmail.com>
Download mbox | patch
Permalink /patch/185516/
State New
Headers show

Comments

Christophe Lyon - Sept. 20, 2012, 10:14 p.m.
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.

Patch

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);
+}