[PATCH] D137768: [opt] Enable using -module-summary with -S

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

Thanks! Any chance you could fix the "clang -flto=thin -S" case, which I believe is handled by the call to PrintModulePass in clang/lib/CodeGen/BackendUtil.cpp? The logic for determining whether we should generate and emit a summary can be found just above the call to PrintModulePass, currently used when emitting bitcode.



================
Comment at: llvm/lib/IR/IRPrintingPasses.cpp:56
+  if (Index) {
+    if (Index->modulePaths().empty())
+      Index->addModule("", 0);
----------------
What is this case for?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137768