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

hfinkel at anl.gov hfinkel at anl.gov
Sat Jul 25 18:27:53 PDT 2015

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?


More information about the llvm-commits mailing list