[PATCH] D47414: [llvm-objcopy] Fix null symbol handling

[Paul Semel via Phabricator via llvm-commits]

paulsemel created this revision.
paulsemel added reviewers: jhenderson, alexshap, echristo, jakehehrlich.

There is no need to expose the null symbol to the user.
This follows the same behavior as how we are handling null section.
This by the same time, fix the bad behavior of `--strip-all` option, where the binary result was malformed (missing zero symbol).


Repository:
  rL LLVM

https://reviews.llvm.org/D47414

Files:
  test/tools/llvm-objcopy/keep-file-symbols.test
  test/tools/llvm-objcopy/strip-all-and-keep-symbol.test
  tools/llvm-objcopy/Object.cpp
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47414.148724.patch
Type: text/x-patch
Size: 5388 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180526/95bbbbae/attachment.bin>