[ELF] Apply version script patterns to non-default version symbols
authorFangrui Song <i@maskray.me>
Wed, 4 Aug 2021 16:02:11 +0000 (09:02 -0700)
committerFangrui Song <i@maskray.me>
Wed, 4 Aug 2021 16:02:11 +0000 (09:02 -0700)
commit7ed22a6fa90cbdc70d6806c1121a0c50c1978dce
tree9677e62f092fb13d08c2542447dfe02237930db6
parent3bc8ce5dd718beef0031bf4b070ac4026e6910d7
[ELF] Apply version script patterns to non-default version symbols

Currently version script patterns are ignored for .symver produced
non-default version (single @) symbols. This makes such symbols
not localizable by `local:`, e.g.

```
.symver foo3_v1,foo3@v1
.globl foo_v1
foo3_v1:

ld.lld --version-script=a.ver -shared a.o
# In a.out, foo3@v1 is incorrectly exported.
```

This patch adds the support:

* Move `config->versionDefinitions[VER_NDX_LOCAL].patterns` to `config->versionDefinitions[versionId].localPatterns`
* Rename `config->versionDefinitions[versionId].patterns` to `config->versionDefinitions[versionId].nonLocalPatterns`
* Allow `findAllByVersion` to find non-default version symbols when `includeNonDefault` is true. (Note: `symtab` keys do not have `@@`)
* Make each pattern check both the unversioned `pat.name` and the versioned `${pat.name}@${v.name}`
* `localPatterns` can localize `${pat.name}@${v.name}`. `nonLocalPatterns` can prevent localization by assigning `verdefIndex` (before `parseSymbolVersion`).

---

If a user notices new `undefined symbol` errors with a version script containing
`local: *;`, the issue is likely due to a missing `global:` pattern.

Reviewed By: peter.smith

Differential Revision: https://reviews.llvm.org/D107234
12 files changed:
lld/ELF/Config.h
lld/ELF/Driver.cpp
lld/ELF/ScriptParser.cpp
lld/ELF/SymbolTable.cpp
lld/ELF/SymbolTable.h
lld/ELF/Symbols.cpp
lld/test/ELF/version-script-extern-exact.s [deleted file]
lld/test/ELF/version-script-extern-wildcards.s [deleted file]
lld/test/ELF/version-script-extern.s [deleted file]
lld/test/ELF/version-script-noundef.s
lld/test/ELF/version-script-symver-extern.s [new file with mode: 0644]
lld/test/ELF/version-script-symver.s