[PATCH] D36113: [Loop Vectorize] Vectorize Loops with Backward Dependence

DIVYA SHANMUGHAN via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 15:02:48 PDT 2017


DIVYA marked 6 inline comments as done.
DIVYA added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorizePred.cpp:152
+  for (auto Dep : *Deps) {
+    if ((Dep.Type == MemoryDepChecker::Dependence::DepType::Backward)) {
+
----------------
fhahn wrote:
> I think you could use Dep.isBackward(), which is shorter.
Dep.isBackward() will take   BackwardVectorizable, Backward and BackwardVectorizableButPreventsForwarding cases into consideration.But Here  its done only for Backward dependences 


https://reviews.llvm.org/D36113





More information about the llvm-commits mailing list