diff options
author | Tom de Vries <tdevries@suse.de> | 2023-10-07 10:33:29 +0200 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2023-10-07 10:33:29 +0200 |
commit | 6542e3df20652b96851c9f801ce199e3072792cf (patch) | |
tree | 7cf49541658bd231a6f460a83a3496671653f437 | |
parent | 7a3bb62d82631ff042525b7115d9e03a7329e7be (diff) | |
download | gdb-6542e3df20652b96851c9f801ce199e3072792cf.zip gdb-6542e3df20652b96851c9f801ce199e3072792cf.tar.gz gdb-6542e3df20652b96851c9f801ce199e3072792cf.tar.bz2 |
[gdb/testsuite] Fix gdb.arch/i386-signal.exp on x86_64
On x86_64-linux, with test-case gdb.arch/i386-signal.exp I run into:
...
builtin_spawn -ignore SIGHUP gcc -fno-stack-protector i386-signal.c \
-fdiagnostics-color=never -fno-pie -g -no-pie -lm -o i386-signal^M
/tmp/cc2xydTG.s: Assembler messages:^M
/tmp/cc2xydTG.s:50: Error: operand size mismatch for `push'^M
compiler exited with status 1
output is:
/tmp/cc2xydTG.s: Assembler messages:^M
/tmp/cc2xydTG.s:50: Error: operand size mismatch for `push'^M
gdb compile failed, /tmp/cc2xydTG.s: Assembler messages:
/tmp/cc2xydTG.s:50: Error: operand size mismatch for `push'
UNTESTED: gdb.arch/i386-signal.exp: failed to compile
...
This is with gas 2.41, it compiles without problems with gas 2.40. Some more
strict checking was added in commit 5cc007751cd ("x86: further adjust
extend-to-32bit-address conditions"). This may or may not be a gas regression
( https://sourceware.org/pipermail/binutils/2023-October/129818.html ).
The offending bit is:
...
" push $sigframe\n"
...
which refers to a function:
...
" .globl sigframe\n"
"sigframe:\n"
...
The test-case passes with target board unix/-m32.
Make the test-case work by using pushq instead of push for the
is_amd64_regs_target case.
Tested on x86_64-linux, with target boards:
- unix/-m64 (is_amd64_regs_target == 1), and
- unix/-m32 (is_amd64_regs_target == 0),
PR testsuite/30928
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30928
-rw-r--r-- | gdb/testsuite/gdb.arch/i386-signal.c | 4 | ||||
-rw-r--r-- | gdb/testsuite/gdb.arch/i386-signal.exp | 7 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gdb/testsuite/gdb.arch/i386-signal.c b/gdb/testsuite/gdb.arch/i386-signal.c index 19bb1bb..4bf97e5 100644 --- a/gdb/testsuite/gdb.arch/i386-signal.c +++ b/gdb/testsuite/gdb.arch/i386-signal.c @@ -45,7 +45,11 @@ asm(".text\n" " .align 8\n" " .globl setup\n" "setup:\n" +#if IS_AMD64_REGS_TARGET + " pushq $sigframe\n" +#else " push $sigframe\n" +#endif " jmp func\n" "\n" " .cfi_startproc\n" diff --git a/gdb/testsuite/gdb.arch/i386-signal.exp b/gdb/testsuite/gdb.arch/i386-signal.exp index f6a8871..a69172e 100644 --- a/gdb/testsuite/gdb.arch/i386-signal.exp +++ b/gdb/testsuite/gdb.arch/i386-signal.exp @@ -19,8 +19,13 @@ require {is_any_target "i?86-*-*" "x86_64-*-*"} standard_testfile +set opts {} +lappend opts debug +lappend opts nopie +lappend opts additional_flags=-DIS_AMD64_REGS_TARGET=[is_amd64_regs_target] + if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ - executable { debug nopie }] != "" } { + executable $opts] != "" } { untested "failed to compile" return -1 } |