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