[PATCH] Disable indvar widening if arithmetics on the wider type are more expensive

Justin Holewinski justin.holewinski at gmail.com
Mon Nov 10 10:10:11 PST 2014


This looks like a good start to me, but I'm not an expert on IndVarSimplify.

================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:78
@@ -77,1 +77,3 @@
+    TargetLibraryInfo         *TLI;
+    const TargetTransformInfo *TTI;
 
----------------
I'm not opposed to the style change, but it is extra noise in the patch.

http://reviews.llvm.org/D6188






More information about the llvm-commits mailing list