Hi Stefan,
On 02/01/19 21:47, Stefan Agner wrote:
Add a test case to check whether -masm-syntax-unified is indeed
emitting the inline assembler with .syntax unified.
Can you please provide a ChangeLog entry for this change.
---
.../gcc.target/arm/pr88648-asm-syntax-unified.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
create mode 100644 gcc/testsuite/gcc.target/arm/pr88648-asm-syntax-unified.c
diff --git a/gcc/testsuite/gcc.target/arm/pr88648-asm-syntax-unified.c
b/gcc/testsuite/gcc.target/arm/pr88648-asm-syntax-unified.c
new file mode 100644
index 00000000000..2bd9d891b9e
--- /dev/null
+++ b/gcc/testsuite/gcc.target/arm/pr88648-asm-syntax-unified.c
@@ -0,0 +1,14 @@
+/* Test for unified syntax assembly generation. */
+/* { dg-do compile } */
+/* { dg-require-effective-target arm_arch_v7a_ok } */
+/* { dg-add-options arm_arch_v7a } */
+/* { dg-options "-marm -march=armv7-a -masm-syntax-unified" } */
+
+void test ()
+{
+ asm("nop");
+}
+
+/* { dg-final { scan-assembler-times {\.syntax\sunified} 3 } } */
+/* { dg-final { scan-assembler-times {\.syntax\sdivided} 0 } } */
+
Please use scan-assembler-not here to check for the absence of the ".syntax
divided".
Looks ok to me otherwise.
Do you need someone to commit these for you?
Thanks,
Kyrill
--
2.20.1
