[PATCH] D28907: [SLP] Fix for PR30787: Failure to beneficially vectorize 'copyable' elements in integer binary ops.

Dinar Temirbulatov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 16 07:13:40 PST 2017


dtemirbulatov added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3729
+      }
+    // Check inner vector dependencies
+    for (Instruction *I1 : Alternatives)
----------------
I have observed some syntectic testcases where we could have cycle dependencies formed, I am sure without this check we could hit the issue somehow.


https://reviews.llvm.org/D28907





More information about the llvm-commits mailing list