[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