[PATCH] D21018: [ELF] - Basic versioned symbols support implemented.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 04:01:32 PDT 2016


grimar updated this revision to Diff 60014.
grimar added a comment.

- Teach version-script parser to parse parent versions declarations. That allows to parse script file from reproduce file posted by Rafael in original issue thread (https://llvm.org/bugs/show_bug.cgi?id=23231).

After that change, I can link the reproduce if add the --allow-multiple-definition flag. Flag helps here as this patch does not support comparing versioned vs unversioned symbols yet. So for example

acl_compat.So contains acl_add_perm at FBSD_1.0
acl_perm.So contains acl_add_perm

Resolving fails without flag with duplicate symbol error now.


http://reviews.llvm.org/D21018

Files:
  ELF/Config.h
  ELF/Driver.cpp
  ELF/InputFiles.cpp
  ELF/OutputSections.cpp
  ELF/OutputSections.h
  ELF/SymbolListFile.cpp
  ELF/SymbolListFile.h
  ELF/SymbolTable.cpp
  ELF/Symbols.h
  ELF/Writer.cpp
  test/ELF/Inputs/verdef.s
  test/ELF/verdef.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21018.60014.patch
Type: text/x-patch
Size: 21263 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160608/0a5e5bb3/attachment.bin>


More information about the llvm-commits mailing list