[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