[PATCH] D30200: [SLP] Fix for PR31880: shuffle and vectorize repeated scalar ops on extracted elements

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 06:12:17 PDT 2017


ABataev marked 2 inline comments as done.
ABataev added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:239
+      Vec1 = Vec;
+    else if (!Vec2)
+      Vec2 = Vec;
----------------
mkuper wrote:
> Shouldn't this be something like
> 
> ```
> else if (Vec1 != Vec)
> ```
> ?
Yes, you're correct, thanks.


https://reviews.llvm.org/D30200





More information about the llvm-commits mailing list