[PATCH] D41684: [llvm-objcopy] Add --localize-hidden option

Jake Ehrlich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 16:08:14 PST 2018


jakehehrlich updated this revision to Diff 128580.
jakehehrlich added a comment.

1. Moved RemovePred decl as recommended
2. Added spaces around a bit of code
3. Made some big changes to the test. I added weak and local hidden symbols. I changed their names because I couldn't keep track of anything. I changed a symbol to be protected. I believe it should meet the requirements originally desired.


Repository:
  rL LLVM

https://reviews.llvm.org/D41684

Files:
  test/tools/llvm-objcopy/localize-hidden.test
  tools/llvm-objcopy/Object.cpp
  tools/llvm-objcopy/Object.h
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41684.128580.patch
Type: text/x-patch
Size: 7013 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180104/50e8a4a7/attachment.bin>


More information about the llvm-commits mailing list