[PATCH] D17353: Set minimum cost of speculating an instruction.

David Li via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 14:59:35 PST 2016


davidxl added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:321
@@ +320,3 @@
+  // Cost of the instruction should be no less than TCC_Basic.
+  if (Cost < TargetTransformInfo::TCC_Basic)
+    Cost = TargetTransformInfo::TCC_Basic;
----------------
Why can't ComputeSpeculationCost return the correct cost here?


http://reviews.llvm.org/D17353





More information about the llvm-commits mailing list