[PATCH] D15401: Refactor threshold computation for inline cost analysis

[David Li via llvm-commits]

davidxl added inline comments.

================
Comment at: lib/Analysis/InlineCost.cpp:197
@@ -164,3 +196,3 @@
   CallAnalyzer(const TargetTransformInfo &TTI, AssumptionCacheTracker *ACT,
                Function &Callee, int Threshold, CallSite CSArg)
     : TTI(TTI), ACT(ACT), F(Callee), CandidateCS(CSArg), Threshold(Threshold),
----------------
A related follow up that can be done:

1) change the parameter name to DefaultThreshold
2) Save the default threshold value in a different member so that it is not overriden by updatethreshold
3) Use the default threshold when recurisively analyizing the indirect call (instead of a constant).

(Not required for this patch).


http://reviews.llvm.org/D15401