[flang-commits] [flang] [Flang][Driver] Emit module summary for Full LTO (PR #164302)

[Tarun Prabhu via flang-commits]

================
@@ -1019,24 +1019,40 @@ void CodeGenAction::runOptimizationPipeline(llvm::raw_pwrite_stream &os) {
 
   // Create the pass manager.
   llvm::ModulePassManager mpm;
-  if (opts.PrepareForFatLTO) {
-    // The module summary should be emitted by default for regular LTO
-    // except for ld64 targets.
-    bool emitSummary = opts.PrepareForThinLTO || opts.PrepareForFullLTO ||
-                       triple.getVendor() != llvm::Triple::Apple;
+  // The module summary should be emitted by default for regular LTO
+  // except for ld64 targets.
+  bool emitSummary = (opts.PrepareForThinLTO || opts.PrepareForFullLTO) &&
----------------
tarunprabhu wrote:

Since ThinLTO support is not being enabled here, should `emitSummary` only be true if `opt.PrepareForFullLTO` is set? Should `PrepareForThinLTO` be ignored here?

https://github.com/llvm/llvm-project/pull/164302