• Austin Clements's avatar
    cmd/link: fix isStmt DWARF info · 8dba66df
    Austin Clements authored
    When CL 147160 introduced function ABIs encoded as symbol versions in
    the linker, it became slightly more complicated to look up derived
    DWARF symbols. It fixed this by introducing a dwarfFuncSym function to
    hide this logic, but missed one derived lookup that was done in the
    object reader itself. As a result, we lost the isStmt tables from the
    compiler, so every PC was marked as a statement in the DWARF info.
    
    Fix this by moving this derived lookup out of the object reader and
    into the DWARF code and calling dwarfFuncSym to get the correctly
    versioned symbol.
    
    Should fix the linux-amd64-longtest builder.
    
    Updates #27539.
    
    Change-Id: If40d5ba28bab1918ac4ad18fbb5103666b6d978b
    Reviewed-on: https://go-review.googlesource.com/c/149605
    Run-TryBot: Austin Clements <austin@google.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarDavid Chase <drchase@google.com>
    8dba66df
Name
Last commit
Last update
..
amd64 Loading commit data...
arm Loading commit data...
arm64 Loading commit data...
ld Loading commit data...
loadelf Loading commit data...
loadmacho Loading commit data...
loadpe Loading commit data...
loadxcoff Loading commit data...
mips Loading commit data...
mips64 Loading commit data...
objfile Loading commit data...
ppc64 Loading commit data...
s390x Loading commit data...
sym Loading commit data...
wasm Loading commit data...
x86 Loading commit data...