[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