aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
authorTom de Vries <tdevries@suse.de>2025-03-18 16:22:02 +0100
committerTom de Vries <tdevries@suse.de>2025-03-18 16:22:02 +0100
commit29b2c50ece6d39ca1ad535961da48eb1ec874c9e (patch)
tree131c3784741c01e110608497a28db6d0b5e78628 /gdb
parent6eb9dab4c99725c1de4bccfeb99e766e7ee657a4 (diff)
downloadbinutils-29b2c50ece6d39ca1ad535961da48eb1ec874c9e.zip
binutils-29b2c50ece6d39ca1ad535961da48eb1ec874c9e.tar.gz
binutils-29b2c50ece6d39ca1ad535961da48eb1ec874c9e.tar.bz2
[gdb/testsuite] Fix gdb.base/enum_cond.exp on arm-linux
On arm-linux, I run into: ... gdb compile failed, ld: warning: enum_cond.o uses variable-size enums yet \ the output is to use 32-bit enums; use of enum values across objects may fail UNTESTED: gdb.base/enum_cond.exp: failed to compile ... Fix this by using -nostdlib. Tested on arm-linux and x86_64-linux. Approved-By: Simon Marchi <simon.marchi@efficios.com>
Diffstat (limited to 'gdb')
-rw-r--r--gdb/testsuite/gdb.base/enum_cond.c16
-rw-r--r--gdb/testsuite/gdb.base/enum_cond.exp15
-rw-r--r--gdb/testsuite/lib/gdb.exp12
3 files changed, 42 insertions, 1 deletions
diff --git a/gdb/testsuite/gdb.base/enum_cond.c b/gdb/testsuite/gdb.base/enum_cond.c
index 35e126a..df8830f 100644
--- a/gdb/testsuite/gdb.base/enum_cond.c
+++ b/gdb/testsuite/gdb.base/enum_cond.c
@@ -46,3 +46,19 @@ main (void)
return 0;
}
+void
+exit (int status)
+{
+#if HAVE_BUILTIN_TRAP
+ __builtin_trap ();
+#endif
+ while (1)
+ ;
+}
+
+void
+_start (void)
+{
+ main ();
+ exit (0);
+}
diff --git a/gdb/testsuite/gdb.base/enum_cond.exp b/gdb/testsuite/gdb.base/enum_cond.exp
index 3ee90f8..d58da47 100644
--- a/gdb/testsuite/gdb.base/enum_cond.exp
+++ b/gdb/testsuite/gdb.base/enum_cond.exp
@@ -19,7 +19,20 @@
standard_testfile .c
-set opts [list debug additional_flags=-fshort-enums]
+set opts {}
+lappend opts debug
+lappend opts additional_flags=-fshort-enums
+# Without -nostdlib, on arm we run into:
+#
+# ld: warning: enum_cond.o uses variable-size enums yet the output is to use
+# 32-bit enums; use of enum values across objects may fail
+#
+# due to conflicting values for Tag_ABI_enum_size between enum_cond.o and
+# linked-in objects. Work around this by using -nostdlib, making sure there's
+# just one object, and no such conflict can happen.
+lappend opts additional_flags=-nostdlib
+lappend opts additional_flags=-DHAVE_BUILTIN_TRAP=[have_builtin_trap]
+
if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable $opts] != "" } {
untested "failed to compile"
return -1
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 481f9ec..3349da7 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -11072,5 +11072,17 @@ proc section_get {exec section} {
return $retval
}
+# Return 1 if the compiler supports __builtin_trap, else return 0.
+
+gdb_caching_proc have_builtin_trap {} {
+
+ return [gdb_can_simple_compile builtin_trap {
+ int main() {
+ __builtin_trap ();
+ return 0;
+ }
+ } executable]
+}
+
# Always load compatibility stuff.
load_lib future.exp