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

[Alex Brachet via Phabricator via llvm-commits]

abrachet marked an inline comment as done.
abrachet 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"),
----------------
haowei wrote:
> Since this flag affects both IFS output as well as ELF output. I think it is better to rename this variable to `StripNeededLibs`
Done. Out of curiosity is there a reason that these other flags are only meaningful when outputting ifs? Why not have strip-undefined strip undefined syms when outputting elf stubs too?


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

https://reviews.llvm.org/D119907