[PATCH] D28708: Compute summary before calling extractProfTotalWeight

[David Li via Phabricator via llvm-commits]

davidxl accepted this revision.
davidxl added a comment.

lgtm



================
Comment at: unittests/Analysis/ProfileSummaryInfoTest.cpp:101
+TEST_F(ProfileSummaryInfoTest, TestNoProfile) {
+  auto M = makeLLVMModule();
+  Function *F = M->getFunction("f");
----------------
It is slightly more readable to do:

const char *ProfKind = nullptr;
auto M = makeLLVMModule(ProfKind);


https://reviews.llvm.org/D28708