[PATCH] D15177: [LoopVectorizer] Refine loop vectorizer's register usage calculator by ignoring specific instructions.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 12 16:11:54 PST 2015


hfinkel added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:5645
@@ +5644,3 @@
+    SmallPtrSetImpl<Instruction *> &Casts = RedDes.getCastInsts();
+    ValuesToIgnore.insert(Casts.begin(), Casts.end());
+  }
----------------
Shouldn't this be VecValuesToIgnore? These casts are not vectorized, but they are scalar instructions that maintain a live value in a register.

(Is this the only contributor to ValuesToIgnore?)


http://reviews.llvm.org/D15177





More information about the llvm-commits mailing list