[PATCH] D49428: [LSV] Look through selects for consecutive addresses

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:140
 
+  enum { RecursionLimit = 3 };
+
----------------
Why is this an enum?


================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:440-441
+
+  if (auto *SelectA = dyn_cast<SelectInst>(PtrA))
+    if (auto *SelectB = dyn_cast<SelectInst>(PtrB))
+      return SelectA->getCondition() == SelectB->getCondition() &&
----------------
Braces


Repository:
  rL LLVM

https://reviews.llvm.org/D49428