This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] RISC-V: Force variables to .data for code_elim.
- From: Jim Wilson <wilson at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 6 Nov 2018 19:07:26 -0000
- Subject: [binutils-gdb] RISC-V: Force variables to .data for code_elim.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=109be305bb6acf7604f95411f333549f33a673bb
commit 109be305bb6acf7604f95411f333549f33a673bb
Author: Jim Wilson <jimw@sifive.com>
Date: Tue Nov 6 11:06:23 2018 -0800
RISC-V: Force variables to .data for code_elim.
RISC-V puts a global variable in .sdata by default, which causes the
add-symbol-file commands with -s .data to fail as there is no .data section.
This fixes 3 testsuite failures.
gdb/testsuite/
* gdb.base/code_elim.exp: For riscv, set additional_flags
to include -msmall-data-limit=0.
Diff:
---
gdb/testsuite/ChangeLog | 5 +++++
gdb/testsuite/gdb.base/code_elim.exp | 5 +++++
2 files changed, 10 insertions(+)
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 0df75aa..24e4c92 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2018-11-06 Jim Wilson <jimw@sifive.com>
+
+ * gdb.base/code_elim.exp: For riscv, set additional_flags
+ to include -msmall-data-limit=0.
+
2018-11-01 Joel Brobecker <brobecker@adacore.com>
* gdb.ada/watch_minus_l: New testcase.
diff --git a/gdb/testsuite/gdb.base/code_elim.exp b/gdb/testsuite/gdb.base/code_elim.exp
index 962df4d..1c76827 100644
--- a/gdb/testsuite/gdb.base/code_elim.exp
+++ b/gdb/testsuite/gdb.base/code_elim.exp
@@ -28,6 +28,11 @@ lappend opts "additional_flags=-fdata-sections"
lappend opts "additional_flags=-Wl,-gc-sections"
lappend opts "additional_flags=-Wl,-e,main"
+# Place variables in .data instead of .sdata.
+if {[istarget "riscv*-*-*"]} {
+ lappend opts "additional_flags=-msmall-data-limit=0"
+}
+
remote_exec build "rm -f ${binfile1}"
remote_exec build "rm -f ${binfile2}"