[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