[PATCH] D9822: Reducing the costs of cast instructions to enable more vectorization of smaller types in LoopVectorize

[hfinkel at anl.gov]

hfinkel added a subscriber: hfinkel.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:1182
@@ +1181,3 @@
+
+  /// Use the map of CandidateNarrowInstrs to confirm the usability of smaller types
+  /// for chains of instructions.
----------------
Line is too long.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:5203
@@ +5202,3 @@
+        // Need to confirm the type of this operand
+        if (ConfirmNarrowChain(NextOp, VF, NarrowTy))
+          ++NumConfirmed;
----------------
samparker wrote:
> jmolloy wrote:
> > This is unboundedly recursive.
> But isn't the recursion bound by the size of CandiateNarrowInstrs, which is defined during the first pass?
What bounds that size?



http://reviews.llvm.org/D9822