[PATCH] D137768: [opt][clang] Enable using -module-summary /-flto=thin with -S / -emit-llvm

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 14 17:41:43 PST 2022


shafik added inline comments.


================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:996
+        MPM.addPass(PrintModulePass(*OS, "", CodeGenOpts.EmitLLVMUseLists,
+                                    true /* EmitLTOSummary */));
+      }
----------------
Nitpick, but we should be consistent with the documented format for clang-tidy bugprone-argument-comment: https://clang.llvm.org/extra/clang-tidy/checks/bugprone/argument-comment.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137768



More information about the cfe-commits mailing list