Page MenuHomePhabricator

[llvm-objcopy] Add --keep-symbols option
ClosedPublic

Authored by kongyi on Mar 31 2019, 8:25 PM.

Details

Summary

We have all the other variants of -keep-*-symbols, but missing --keep-symbols.

Diff Detail

Repository
rL LLVM

Event Timeline

kongyi created this revision.Mar 31 2019, 8:25 PM

Code looks good, but can you add a test for this? Adding something to test/tools/llvm-objcopy/ELF/keep-symbol.test should be fine

kongyi updated this revision to Diff 193108.Apr 1 2019, 9:36 AM

Added tests

rupprecht accepted this revision.Apr 1 2019, 10:48 AM
This revision is now accepted and ready to land.Apr 1 2019, 10:48 AM
This revision was automatically updated to reflect the committed changes.