[PATCH] D119907: [ifs] Add --strip-needed flag

Haowei Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 16 10:09:59 PST 2022


haowei added inline comments.


================
Comment at: llvm/tools/llvm-ifs/llvm-ifs.cpp:106
                    cl::cat(IfsCategory));
+cl::opt<bool> StripIFSNeededLibs("strip-needed",
+                                 cl::desc("Strip needed libs from IFS output"),
----------------
Since this flag affects both IFS output as well as ELF output. I think it is better to rename this variable to `StripNeededLibs`


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

https://reviews.llvm.org/D119907



More information about the llvm-commits mailing list