LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 30312 - Only one of multiple mangled symbols matching a version script entry has the symbol version appended
Summary: Only one of multiple mangled symbols matching a version script entry has the ...
Status: RESOLVED FIXED
Alias: None
Product: lld
Classification: Unclassified
Component: ELF (show other bugs)
Version: unspecified
Hardware: PC FreeBSD
: P normal
Assignee: George Rimar
URL:
Keywords:
Depends on:
Blocks: 23214
  Show dependency tree
 
Reported: 2016-09-07 13:01 PDT by emaste
Modified: 2016-11-07 14:03 PST (History)
3 users (show)

See Also:
Fixed By Commit(s):

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description emaste 2016-09-07 13:01:32 PDT 
I've added a change to a test case to demonstrate the issue in https://reviews.llvm.org/D24306

Both of these symbols demangle to abc::abc():
_ZN3abcC1Ev
_ZN3abcC2Ev

(These would be abc's complete object constructor and base object constructor, respectively.)

however with `"abc::abc()"` in the version script only one of the two receives the symbol version.

From llvm-readobj -V -dyn-symbols I find these two symbols:

_ZN3abcC1Ev@
_ZN3abcC2Ev@@LIBSAMPLE_1.0
Comment 1 George Rimar 2016-09-08 05:35:28 PDT 
Patch: https://reviews.llvm.org/D24336
Comment 2 George Rimar 2016-09-13 05:58:31 PDT 
r281318
Comment 3 emaste 2016-09-14 13:39:29 PDT 
Reopened as this change was reverted in r281411 due to buildbot failures.

FWIW it passes on my FreeBSD 10.3 machine.
Comment 4 emaste 2016-09-15 09:04:09 PDT 
Recommitted as r281605
Comment 5 George Rimar 2016-09-15 09:23:32 PDT 
Yes, but still waiting for build bot, so a bit too early, lets hope it is fine )

(http://lab.llvm.org:8011/builders/clang-with-lto-ubuntu)