* [PATCH] ARM: add test case for -masm-syntax-unified (PR88648)
@ 2019-01-02 21:48 Stefan Agner
2019-01-08 9:35 ` Kyrill Tkachov
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Agner @ 2019-01-02 21:48 UTC (permalink / raw)
To: nickc, richard.earnshaw, ramana.radhakrishnan, kyrylo.tkachov
Cc: gcc-patches, Stefan Agner
Add a test case to check whether -masm-syntax-unified is indeed
emitting the inline assembler with .syntax unified.
---
.../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 } } */
+
--
2.20.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: add test case for -masm-syntax-unified (PR88648)
2019-01-02 21:48 [PATCH] ARM: add test case for -masm-syntax-unified (PR88648) Stefan Agner
@ 2019-01-08 9:35 ` Kyrill Tkachov
2019-01-08 21:56 ` Stefan Agner
0 siblings, 1 reply; 3+ messages in thread
From: Kyrill Tkachov @ 2019-01-08 9:35 UTC (permalink / raw)
To: Stefan Agner, nickc, Richard Earnshaw, Ramana Radhakrishnan; +Cc: gcc-patches
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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: add test case for -masm-syntax-unified (PR88648)
2019-01-08 9:35 ` Kyrill Tkachov
@ 2019-01-08 21:56 ` Stefan Agner
0 siblings, 0 replies; 3+ messages in thread
From: Stefan Agner @ 2019-01-08 21:56 UTC (permalink / raw)
To: Kyrill Tkachov; +Cc: nickc, Richard Earnshaw, Ramana Radhakrishnan, gcc-patches
On 08.01.2019 10:35, Kyrill Tkachov wrote:
> 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".
Ok, will send a v2 soon.
>
> Looks ok to me otherwise.
> Do you need someone to commit these for you?
Yes please, I do not have commit rights.
--
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-01-08 21:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-02 21:48 [PATCH] ARM: add test case for -masm-syntax-unified (PR88648) Stefan Agner
2019-01-08 9:35 ` Kyrill Tkachov
2019-01-08 21:56 ` Stefan Agner
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).