[patch, testsuite] Patch for PR 43283, semicolon may not start new line

Steve Ellcey sje@cup.hp.com
Thu Jun 24 18:03:00 GMT 2010


The gcc.dg/lto/20091216-1 test fails on PA and IA64 platforms for two
reasons.  One is that a semicolon starts a comment on these platforms
and not a new line.  Other platforms probably have this issue too based
on the value of comment_chars in the GNU assembler sources. The second
issue is that the nop statement on IA64 requires an argument.  This
patch fixes both problems.  I made the change from a semicolon to a
newline global but it could be ifdefed on __hppa if people don't want
the change to be global.  This seems to work fine for me though on x86
linux as well as my PA and IA64 platforms.

Tested on hppa2.0w-hp-hpux11.11, hppa64-hp-hpux11.11, ia64-hp-hpux11.23,
ia64-debian-linux-gnu, and x86_64-redhat-linux-gnu.

OK for checkin?

Steve Ellcey
sje@cup.hp.com



2010-06-24  Steve Ellcey  <sje@cup.hp.com>

	PR testsuite/43283
	* gcc.dg/lto/20091216-1_0.c: Use newline instead of semicolon
	and add argument to nop for IA64.


Index: gcc.dg/lto/20091216-1_0.c
===================================================================
--- gcc.dg/lto/20091216-1_0.c	(revision 161306)
+++ gcc.dg/lto/20091216-1_0.c	(working copy)
@@ -1,6 +1,10 @@
 /* { dg-lto-do run } */
 
-asm (".globl start_; start_: nop");
+#ifdef __ia64
+asm (".globl start_\nstart_: nop 0");
+#else
+asm (".globl start_\nstart_: nop");
+#endif
 
 int
 main ()



More information about the Gcc-patches mailing list