[PATCH] D18289: Attach profile summary information to module

Easwaran Raman via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 22 14:37:35 PDT 2016


eraman marked 2 inline comments as done.

================
Comment at: lib/CodeGen/CodeGenModule.cpp:398-399
@@ -397,3 +397,4 @@
   if (PGOReader) {
     getModule().setMaximumFunctionCount(PGOReader->getMaximumFunctionCount());
+    getModule().setProfileSummary(PGOReader->getSummary().getMD(VMContext));
     if (PGOStats.hasDiagnostics())
----------------
The previous line? Yes.

================
Comment at: test/Profile/profile-summary.c:6
@@ +5,3 @@
+//
+int begin(int i) {
+  if (i)
----------------
I didn't see David's comment and your subseqquent response. I have changed the test to only check for the presence of Profilesummary and DetailedSummary fields. 


http://reviews.llvm.org/D18289





More information about the cfe-commits mailing list