[PATCH] D57144: [SLPVectorizer] Get rid of IndexQueue array from vectorizeStores. NFCI.

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 24 23:50:10 PST 2019


yrouban marked 3 inline comments as done.
yrouban added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4749
 
+  auto FindConsecutiveAccess = [&] (int K, int Idx) {
+    if (!isConsecutiveAccess(Stores[K], Stores[Idx], *DL, *SE))
----------------
ABataev wrote:
> 1. Use `auto &&`.
> 2. Do not use default capture, better to use explicit capture list.
any reference on why //auto &&'// differs?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57144/new/

https://reviews.llvm.org/D57144





More information about the llvm-commits mailing list