[binutils-gdb] Re: ld: testsuite: Add sframe test for PR 33401

Alan Modra amodra@sourceware.org
Sat Dec 13 00:23:21 GMT 2025


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ba038bba603d18b431ad1b69505553881b8a39c7

commit ba038bba603d18b431ad1b69505553881b8a39c7
Author: Alan Modra <amodra@gmail.com>
Date:   Sat Dec 13 09:30:46 2025 +1030

    Re: ld: testsuite: Add sframe test for PR 33401
    
    If user CXXFLAGS include -fno-inline the test fails with
    FAIL: PR ld/33401 (Step 1: Create relocatable object and check R_*_NONE)
    This should not be a FAIL.  If the compiler does not generate R_*_NONE
    (a good thing!) then it should be UNTESTED, which means the readelf
    check can't be done by run_cc_link_tests.  I haven't made that change
    in this patch, just worked around a user -fno-inline.

Diff:
---
 ld/testsuite/ld-sframe/sframe.exp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ld/testsuite/ld-sframe/sframe.exp b/ld/testsuite/ld-sframe/sframe.exp
index acd52a4ef18..5358cfd8184 100644
--- a/ld/testsuite/ld-sframe/sframe.exp
+++ b/ld/testsuite/ld-sframe/sframe.exp
@@ -40,7 +40,7 @@ proc check_pr33401 { } {
     set test_list {
 	{ "PR ld/33401 (Step 1: Create relocatable object and check R_*_NONE)"
 	    "-r"
-	    "-g -O2 -w -Wa,--gsframe"
+	    "-g -O2 -finline -w -Wa,--gsframe"
 	    {StateClient.cpp StatePlaying.cpp}
 	    {{readelf "-r -W" pr33401.rd}}
 	    "pr33401.o"


More information about the Binutils-cvs mailing list