[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