[PATCH] D18560: [TTI] Add getInliningThresholdMultiplier.

[Matt Arsenault via llvm-commits]

arsenm added inline comments.

================
Comment at: lib/Analysis/TargetTransformInfo.cpp:71-73
@@ +70,5 @@
+    const Function *Caller) const {
+  double Multiplier = TTIImpl->getInliningThresholdMultiplier(Caller);
+  assert(!std::isnan(Multiplier) && "Multiplier cannot be nan");
+  assert(!std::isinf(Multiplier) && "Multiplier cannot be infinity");
+  return Multiplier;
----------------
I thought using host FP was to be avoided, so I'm surprised this isn't an integer factor. Couldn't you have one assert on isnormal()?


http://reviews.llvm.org/D18560