[PATCH] D108428: [ifs] Add option to hide undefined symbols

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 24 15:11:33 PDT 2021


phosek added inline comments.


================
Comment at: llvm/tools/llvm-ifs/llvm-ifs.cpp:103
+cl::opt<bool>
+    StripUndefinedSymbol("strip-undefined-symbol",
+                         cl::desc("Strip undefined symbols from IFS output"),
----------------
Small suggestion, I'd call this just `strip-undefined`, the symbol part is already implied and it better matches spelling used by tools like `llvm-objcopy`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108428/new/

https://reviews.llvm.org/D108428



More information about the llvm-commits mailing list