From patchwork Mon Nov 5 14:49:24 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [committed] Fix up pr54970.c testcase Date: Mon, 05 Nov 2012 04:49:24 -0000 From: Jakub Jelinek X-Patchwork-Id: 197207 Message-Id: <20121105144924.GL1881@tucnak.redhat.com> To: gcc-patches@gcc.gnu.org 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 PR debug/54970 PR debug/54971 * gcc.dg/guality/pr54970.c: Use NOP instead of "NOP" in inline-asm. Jakub --- 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; }