• David Crawshaw's avatar
    cmd/link: only exclude C-only symbols on darwin · 9cf06ed6
    David Crawshaw authored
    C-only symbols are excluded from pclntab because of a quirk of darwin,
    where functions are referred to by an exported symbol so dynamic
    relocations de-duplicate to the host binary module and break unwinding.
    
    This doesn't happen on ELF systems because the linker always refers to
    unexported module-local symbols, so we don't need this condition.
    And the current logic for excluding some functions breaks the module
    verification code in moduledataverify1. So disable this for plugins
    on linux.
    
    (In 1.9, it will probably be necessary to introduce a module-local
    symbol reference system on darwin to fix a different bug, so all of
    this onlycsymbol code made be short-lived.)
    
    With this CL, the tests in CL 35116 pass.
    
    Change-Id: I517d7ca4427241fa0a91276c462827efb9383be9
    Reviewed-on: https://go-review.googlesource.com/35190Reviewed-by: 's avatarMichael Hudson-Doyle <michael.hudson@canonical.com>
    Run-TryBot: David Crawshaw <crawshaw@golang.org>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    9cf06ed6
Name
Last commit
Last update
.github Loading commit data...
api Loading commit data...
doc Loading commit data...
lib/time Loading commit data...
misc Loading commit data...
src Loading commit data...
test Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
AUTHORS Loading commit data...
CONTRIBUTING.md Loading commit data...
CONTRIBUTORS Loading commit data...
LICENSE Loading commit data...
PATENTS Loading commit data...
README.md Loading commit data...
favicon.ico Loading commit data...
robots.txt Loading commit data...