[PATCH] D32768: [PM] Add ProfileSummaryAnalysis as a required pass in the new pipeline.

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: lib/Passes/PassBuilder.cpp:504-507
+  // Require the GlobalsAA analysis and ProfileSummaryAnalysis for the module
+  // so we can query them within the CGSCC pipeline.
   MPM.addPass(RequireAnalysisPass<GlobalsAA, Module>());
+  MPM.addPass(RequireAnalysisPass<ProfileSummaryAnalysis, Module>());
----------------
Please add a comment explanining why we require it.


https://reviews.llvm.org/D32768