[PATCH] D22535: [LSV] Don't assume that loads/stores appear in address order in the BB.
Alina Sbirlea via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 19 17:11:37 PDT 2016
asbirlea added inline comments.
================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:455
@@ -451,1 +454,3 @@
+ unsigned ChainInstrLoc = ChainInstrs[ChainInstrIdx].second;
+ bool OK = true;
for (auto EntryMem : MemoryInstrs) {
----------------
Nit: %s/OK/No[Interfering]Alias[Found]
Better yet, reverse the condition to AliasFound = false; and check below
```
if(AliasFound) ...
```
https://reviews.llvm.org/D22535
More information about the llvm-commits
mailing list