[binutils-gdb] Re: ld: Add --enable-memory-seal configure option
Alan Modra
amodra@sourceware.org
Wed Jan 15 03:17:40 GMT 2025
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=e0e6aa08d78372591208733c1247c378cb4bbc35
commit e0e6aa08d78372591208733c1247c378cb4bbc35
Author: Alan Modra <amodra@gmail.com>
Date: Wed Jan 15 13:32:00 2025 +1030
Re: ld: Add --enable-memory-seal configure option
Commit 80dc29527ff9 accidentally removed an assignment to board_flags,
resulting in tcl errors 'can't read "board_flags": no such variable'
on sh4-linux-gnu. Fix that by calling [get_board_flags] in the
condition rather than reinstating the removed line since it seems most
configurations don't have a null STATIC_LDFLAGS. Do the same in
another similar test too.
Diff:
---
ld/testsuite/lib/ld-lib.exp | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/ld/testsuite/lib/ld-lib.exp b/ld/testsuite/lib/ld-lib.exp
index 3fd445b37b8..74125ac1983 100644
--- a/ld/testsuite/lib/ld-lib.exp
+++ b/ld/testsuite/lib/ld-lib.exp
@@ -794,9 +794,8 @@ proc run_ld_link_exec_tests { ldtests args } {
pass $testname
continue;
} else {
- set board_flags [get_board_flags]
if { [string match "" $STATIC_LDFLAGS] \
- && [regexp -- ".* \[-\]+static .*" " $board_flags $ld_options $objfiles $ld_after "] } {
+ && [regexp -- ".* \[-\]+static .*" " [get_board_flags] $ld_options $objfiles $ld_after "] } {
untested $testname
continue
}
@@ -987,7 +986,7 @@ proc run_cc_link_tests { ldtests } {
}
if { [string match "" $STATIC_LDFLAGS] \
- && [regexp -- ".* \[-\]+static .*" " $board_flags $ldflags $objfiles "] } {
+ && [regexp -- ".* \[-\]+static .*" " [get_board_flags] $ldflags $objfiles "] } {
untested $testname
continue
}
More information about the Binutils-cvs
mailing list