From: Jens Remus Date: Fri, 21 Mar 2025 14:29:03 +0000 (+0100) Subject: ld: Correct test pr19719 naming mix-up X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a8373e160b7912628ab70f97c5a336d2375480b3;p=thirdparty%2Fbinutils-gdb.git ld: Correct test pr19719 naming mix-up The suffix "defined/undefined" in the ld test pr19719 name specifies whether weak fun() is defined or undefined is mixed up. The test builds an executable and a shared library. The latter in two flavors, one with weak fun() defined (libpr19719a.so, "defined") and one without weak fun() defined (libpr19719b.so, "undefined"). The first "Run $exe fun [...]" invocation uses libpr19719b.so as libpr19719.so, which is build from dummy.c, which does not define fun. Thus fun is undefined during this test run. The second "Run $exe fun [...]" invocation uses libpr19719a.so as libpr19719.so, which is build from pr19719d.c, which does define fun. Thus fun is defined during this test run. Correct the test naming mix-up accordingly. ld/testsuite/ * ld-elf/shared.exp (mix_pic_and_non_pic): Correct test naming mix-up of when weak fun is un-/defined. Signed-off-by: Jens Remus --- diff --git a/ld/testsuite/ld-elf/shared.exp b/ld/testsuite/ld-elf/shared.exp index bf8952a843c..48fdc542e11 100644 --- a/ld/testsuite/ld-elf/shared.exp +++ b/ld/testsuite/ld-elf/shared.exp @@ -1664,7 +1664,7 @@ proc mix_pic_and_non_pic {xfails cflags ldflags exe} { run_ld_link_exec_tests [list \ [list \ - "Run $exe fun defined" \ + "Run $exe fun undefined" \ "-Wl,--no-as-needed,-rpath,tmpdir $ldflags tmpdir/libpr19719b.o tmpdir/libpr19719.so" \ "" \ { pr19719a.c pr19719c.c } \ @@ -1682,15 +1682,15 @@ proc mix_pic_and_non_pic {xfails cflags ldflags exe} { } if ![isnative] { - unsupported "Run $exe fun undefined" + unsupported "Run $exe fun defined" return } set exec_output [run_host_cmd "tmpdir/$exe" ""] if {![string match "PASS\n" $exec_output]} { - fail "Run $exe fun undefined" + fail "Run $exe fun defined" } else { - pass "Run $exe fun undefined" + pass "Run $exe fun defined" } }