[PATCH] D57440: [llvm-strip] Add --strip-symbol
Jordan Rupprecht via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 30 06:56:47 PST 2019
rupprecht added a comment.
This option exists in objcopy too, can you add it to ObjcopyOpts.td and handle it in the other args parsing block too?
================
Comment at: tools/llvm-objcopy/StripOpts.td:55
+defm strip_symbol : Eq<"strip-symbol", "Strip <symbol>">,
+ MetaVarName<"symbol">;
+def N : JoinedOrSeparate<["-"], "N">, Alias<strip_symbol>;
----------------
Nit: looks like this isn't indented right, it should align w/ Eq
Note: clang-format (from trunk) can fix these simple tablegen files, with the exception that it puts in spaces around `["-"]` that you have to revert -- try `git-clang-format --extensions td --binary ~/build/clang-format master` (I'm trying to submit a couple patches to fix that)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57440/new/
https://reviews.llvm.org/D57440
More information about the llvm-commits
mailing list