[committed] Fix up pr54970.c testcase

Jakub Jelinek jakub@redhat.com
Mon Nov 5 14:49:00 GMT 2012


Hi!

I've noticed that while I'm including ../nop.h header, I was using "NOP"
and so the inclusion was useless.  It makes a difference only on
ia64/s390*/mmix where I didn't test, anyway, this should fix that.
Committed as obvious.

2012-11-05  Jakub Jelinek  <jakub@redhat.com>

	PR debug/54970
	PR debug/54971
	* gcc.dg/guality/pr54970.c: Use NOP instead of "NOP" in inline-asm.

--- gcc.dg/guality/pr54970.c	(revision 193161)
+++ gcc.dg/guality/pr54970.c	(working copy)
@@ -12,28 +12,28 @@ main ()
   int *p = a + 2;		/* { dg-final { gdb-test 15 "a\[1\]" "2" } } */
   int *q = a + 1;		/* { dg-final { gdb-test 15 "a\[2\]" "3" } } */
 				/* { dg-final { gdb-test 15 "*p" "3" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 15 "*q" "2" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 15 "*q" "2" } } */
   *p += 10;			/* { dg-final { gdb-test 20 "a\[0\]" "1" } } */
 				/* { dg-final { gdb-test 20 "a\[1\]" "2" } } */
 				/* { dg-final { gdb-test 20 "a\[2\]" "13" } } */
 				/* { dg-final { gdb-test 20 "*p" "13" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 20 "*q" "2" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 20 "*q" "2" } } */
   *q += 10;			/* { dg-final { gdb-test 25 "a\[0\]" "1" } } */
 				/* { dg-final { gdb-test 25 "a\[1\]" "12" } } */
 				/* { dg-final { gdb-test 25 "a\[2\]" "13" } } */
 				/* { dg-final { gdb-test 25 "*p" "13" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 25 "*q" "12" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 25 "*q" "12" } } */
   __builtin_memcpy (&a, (int [3]) { 4, 5, 6 }, sizeof (a));
 				/* { dg-final { gdb-test 31 "a\[0\]" "4" } } */
 				/* { dg-final { gdb-test 31 "a\[1\]" "5" } } */
 				/* { dg-final { gdb-test 31 "a\[2\]" "6" } } */
 				/* { dg-final { gdb-test 31 "*p" "6" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 31 "*q" "5" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 31 "*q" "5" } } */
   *p += 20;			/* { dg-final { gdb-test 36 "a\[0\]" "4" } } */
 				/* { dg-final { gdb-test 36 "a\[1\]" "5" } } */
 				/* { dg-final { gdb-test 36 "a\[2\]" "26" } } */
 				/* { dg-final { gdb-test 36 "*p" "26" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 36 "*q" "5" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 36 "*q" "5" } } */
   *q += 20;			/* { dg-final { gdb-test 45 "a\[0\]" "4" } } */
 				/* { dg-final { gdb-test 45 "a\[1\]" "25" } } */
 				/* { dg-final { gdb-test 45 "a\[2\]" "26" } } */
@@ -42,6 +42,6 @@ main ()
 				/* { dg-final { gdb-test 45 "p\[-2\]" "4" } } */
 				/* { dg-final { gdb-test 45 "q\[-1\]" "4" } } */
 				/* { dg-final { gdb-test 45 "q\[1\]" "26" } } */
-  asm volatile ("NOP");		/* { dg-final { gdb-test 45 "*q" "25" } } */
+  asm volatile (NOP);		/* { dg-final { gdb-test 45 "*q" "25" } } */
   return 0;
 }

	Jakub



More information about the Gcc-patches mailing list