Bug 32807 - x86-64 test pr19636-3d is not run and would fail
Summary: x86-64 test pr19636-3d is not run and would fail
Status: RESOLVED FIXED
Alias: None
Product: binutils
Classification: Unclassified
Component: ld (show other bugs)
Version: 2.45
: P2 minor
Target Milestone: 2.45
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-03-20 11:59 UTC by Jens Remus
Modified: 2025-03-20 15:48 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:
Project(s) to access:
ssh public key:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jens Remus 2025-03-20 11:59:43 UTC
While experimenting with porting x86-64 commit aec6b87e0b66 ("[x86] Resolve non-PIC undefweak symbols in executable") to s390 I noticed that linker test ld-x86-64/pr19636-3d.d was added but not listed in ld-x86-64/x86-64.exp with subject commit.  Git pickaxe does not report it ever been listed:

$ git log -S"pr19636-3d" -- ld/testsuite/ld-x86-64


Adding it to ld-x86-64/x86-64.exp as follow causes this test to fail in the same way it would on s390:

diff --git a/ld/testsuite/ld-x86-64/x86-64.exp b/ld/testsuite/ld-x86-64/x86-64.exp
--- a/ld/testsuite/ld-x86-64/x86-64.exp
+++ b/ld/testsuite/ld-x86-64/x86-64.exp
@@ -727,6 +727,7 @@ run_dump_test "pr19636-2m"
 run_dump_test "pr19636-3a"
 run_dump_test "pr19636-3b"
 run_dump_test "pr19636-3c"
+run_dump_test "pr19636-3d"
 run_dump_test "pr19645"
 run_dump_test "pr19609-1a"
 run_dump_test "pr19609-1b"

Excerpt from ld.log:

extra regexps in /home/jremus/git/binutils/ld/testsuite/ld-x86-64/pr19636-3d.d starting with "^Symbol table '\.dynsym' contains [0-9]+ entries:$"
EOF from tmpdir/dump.out
FAIL: ld-x86-64/pr19636-3d
Comment 1 Sourceware Commits 2025-03-20 15:47:42 UTC
The master branch has been updated by H.J. Lu <hjl@sourceware.org>:

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

commit 37c6eb16164a2345484777bb5affab45ab502f6f
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Thu Mar 20 08:43:56 2025 -0700

    x86-64: Remove the unused pr19636-3d.d
    
    Remove the unused pr19636-3d.d since static Position Dependent Executable
    doesn't have a dynamic symbol table.
    
            PR ld/32807
            * testsuite/ld-x86-64/pr19636-3d.d: Removed.
    
    Signed-off-by: H.J. Lu <hjl.tools@gmail.com>
Comment 2 H.J. Lu 2025-03-20 15:48:13 UTC
Fixed for 2.45.