[PATCH] D46341: [llvm-objcopy] Add --discard-all (-x) option
Paul Semel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 2 05:44:21 PDT 2018
paulsemel added inline comments.
================
Comment at: tools/llvm-objcopy/Opts.td:76
+def discard_all : Flag<["-", "--"], "discard-all">,
+ HelpText<"Remove all non-global symbols">;
+def x : Flag<["-"], "x">,
----------------
jhenderson wrote:
> paulsemel wrote:
> > jhenderson wrote:
> > > I know this is what gnu objcopy says, but non-global means not STB_GLOBAL to me, but it doesn't affect some other symbol types either (i.e. STB_WEAK, STT_FILE, STT_SECTION). Could you change the help text to match what it actually does, please.
> > Yes, I know, this is a really bad help text. But I was out of ideas, as the real behavior of this option is really tricky... Any thoughts about it guys ?
> > I was thinking of "Remove all but file and section local symbols" , but even for me that sounds really bad to my ears, so..
> How about "remove all local symbols except file and section symbols"? Similar to yours but feels a little less clunky.
I like this one :)
Repository:
rL LLVM
https://reviews.llvm.org/D46341
More information about the llvm-commits
mailing list