[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