This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA]: patch to ia64.inc in gdb.asm testsuite


The attached patch is needed to get the gdb.asm testsuite running successfully for ia64. A recent patch changed the link step to use the target linker rather than gcc. The old method was defaulting dynamic loading. The new code causes a static executable to be created with the first insn being that of gdbasm_startup. Previously, the first insn was just a mov, but this should be an alloc insn. Without this change, a SIGILL is caused for the first insn and the tests all fail.

Ok to commit?

-- Jeff J.


2004-02-04 Jeff Johnston <jjohnstn@redhat.com>


       * gdb.asm/ia64.inc (gdbasm_startup): Make sure first instruction
       executed is an alloc instruction.


Index: ia64.inc
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.asm/ia64.inc,v
retrieving revision 1.1
diff -u -r1.1 ia64.inc
--- ia64.inc	25 Aug 2003 23:43:40 -0000	1.1
+++ ia64.inc	5 Feb 2004 00:34:57 -0000
@@ -36,6 +36,7 @@
 
 	comment "crt0 startup"
 	.macro gdbasm_startup
+	gdbasm_enter
 	mov r32=r0
 	nop.i 0
 	nop.i 0

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]