[PATCH] D13755: Pass to compute dynamic instruction count

David Li via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 10:47:46 PDT 2015


davidxl added inline comments.

================
Comment at: include/llvm/Analysis/InlineCost.h:162
@@ +161,3 @@
+
+  // The call cost analyzer can be used to compute the cost of out-of-line
+  // functions as well. This flag tells whether it is used for inlining or
----------------
Can you explain why it is needed for out-of-line functions -- basically explain how DynamicInstructionCountAnalysis is going to be used. Also why can't the use of frequency/profile information be baked into  CallAnalyzer itself?


Repository:
  rL LLVM

http://reviews.llvm.org/D13755





More information about the llvm-commits mailing list