[PATCH] D34312: [NewPM/Inliner] Reduce threshold for cold callsites in the non-PGO case

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 22:40:05 PDT 2017


davidxl added inline comments.


================
Comment at: lib/Analysis/InlineCost.cpp:691
           Threshold = MinIfValid(Threshold, Params.ColdCallSiteThreshold);
+        } else if (!PSI->hasProfileSummary()) {
+          auto CallSiteBB = CS.getInstruction()->getParent();
----------------
Can the static/local call site check and the global cold site check be moved into a helper function such as CallAnalyzer::isColdCallsite ?


================
Comment at: test/Transforms/Inline/inline-cold-callsite.ll:35
+
+if.then:
+; CHECK: call void @callee
----------------
change the label name to 'cold:'.


https://reviews.llvm.org/D34312





More information about the llvm-commits mailing list