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

Sam Parker sam.parker at arm.com
Mon Jul 20 06:14:02 PDT 2015


samparker added inline comments.

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


http://reviews.llvm.org/D9822







More information about the llvm-commits mailing list