Hookup ProfileSummary with SampleProfilerLoader

[danielcdh]

Summary

Set ProfileSummary in SampleProfilerLoader.

[eraman]

Please add a test case similar to test/Transforms/PGOProfile/PR28219.ll

[danielcdh]

add testcase

[eraman]

The test should be in test/Transforms/SampleProfile/ . I pointed to the test case for reference. You don't need to add a new test case. You can modify an existing test case to test that the summary is available.

[danielcdh]

move test

[eraman]

lgtm

lib/Transforms/IPO/SampleProfile.cpp

for (const auto &I : Reader->getProfiles())
     TotalCollectedSamples += I.second.getTotalSamples();
 
   bool retval = false;
   for (auto &F : M)
     if (!F.isDeclaration()) {
       clearFunctionData();
       retval |= runOnFunction(F);
     }
+  M.setProfileSummary(Reader->getSummary().getMD(M.getContext()));
   return retval;
 }
 
 bool SampleProfileLoaderLegacyPass::runOnModule(Module &M) {
   // FIXME: pass in AssumptionCache correctly for the new pass manager. 
   SampleLoader.setACT(&getAnalysis<AssumptionCacheTracker>());
   return SampleLoader.runOnModule(M);
 }

test/Transforms/SampleProfile/Inputs/summary.prof

+bar:100:3
+ 1: 100
+foo:200:1
+ 1: 200

test/Transforms/SampleProfile/summary.ll

+; Test that we annotate entire program's summary to IR.
+; RUN: opt < %s -sample-profile -sample-profile-file=%S/Inputs/summary.prof -S | FileCheck %s
+
+define i32 @bar() #0 !dbg !1 {
+entry:
+  ret i32 1, !dbg !2
+}
+
+; CHECK-DAG: {{![0-9]+}} = !{i32 1, !"ProfileSummary", {{![0-9]+}}}
+; CHECK-DAG: {{![0-9]+}} = !{!"NumFunctions", i64 2}
+; CHECK-DAG: {{![0-9]+}} = !{!"MaxFunctionCount", i64 3}
+
+!1 = distinct !DISubprogram(name: "bar")
+!2 = !DILocation(line: 2, scope: !2)