]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Pull in r365760 from upstream lld trunk (by Fangrui Song):
authordim <dim@FreeBSD.org>
Sat, 13 Jul 2019 15:04:30 +0000 (15:04 +0000)
committerdim <dim@FreeBSD.org>
Sat, 13 Jul 2019 15:04:30 +0000 (15:04 +0000)
commitacdff3ca6471de660cab42a8c882ac029497dc8b
tree3a2cde69fa79f1e318f9a3918dca9f0ca1443d0e
parent2449e50de0f66dd0ec024c9ed6099597befb2ee6
Pull in r365760 from upstream lld trunk (by Fangrui Song):

  [ELF] Handle non-glob patterns before glob patterns in version
  scripts & fix a corner case of --dynamic-list

  This fixes PR38549, which is silently accepted by ld.bfd.
  This seems correct because it makes sense to let non-glob patterns
  take precedence over glob patterns.

  lld issues an error because
  `assignWildcardVersion(ver, VER_NDX_LOCAL);` is processed before
  `assignExactVersion(ver, v.id, v.name);`.

  Move all assignWildcardVersion() calls after assignExactVersion()
  calls to fix this.

  Also, move handleDynamicList() to the bottom. computeBinding() called
  by includeInDynsym() has this cryptic rule:

      if (versionId == VER_NDX_LOCAL && isDefined() && !isPreemptible)
return STB_LOCAL;

  Before the change:

  * foo's version is set to VER_NDX_LOCAL due to `local: *`
  * handleDynamicList() is called
    - foo.computeBinding() is STB_LOCAL
    - foo.includeInDynsym() is false
    - foo.isPreemptible is not set (wrong)
  * foo's version is set to V1

  After the change:

  * foo's version is set to VER_NDX_LOCAL due to `local: *`
  * foo's version is set to V1
  * handleDynamicList() is called
    - foo.computeBinding() is STB_GLOBAL
    - foo.includeInDynsym() is true
    - foo.isPreemptible is set (correct)

  Reviewed By: ruiu

  Differential Revision: https://reviews.llvm.org/D64550

This makes it longer necessary to patch the version scripts for the
samba ports, to avoid "duplicate symbol 'pdb_search_init' in version
script" errors.

PR: 230602
MFC after: 3 days
contrib/llvm/tools/lld/ELF/SymbolTable.cpp
contrib/llvm/tools/lld/ELF/SymbolTable.h