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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 11:31:43 PST 2017


ABataev added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:1547
+    }
+    if (SameOrAlt < (VL.size()/2))
+      return;
----------------
1. Not formatted
2. I think it is better to make it `<=`


https://reviews.llvm.org/D28907





More information about the llvm-commits mailing list