[PATCH] D109443: [LV] Lazy creation of runtime checks

[Evgeniy via Phabricator via llvm-commits]

ebrevnov added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10052
 
+  GeneratedRTChecks RTChecks(PSE, *LVL, DT, LI, L,
+                             F->getParent()->getDataLayout());
----------------
fhahn wrote:
> Do this need to be moved?
Not strictly necessary for this patch. Last patch in the series makes LoopVectorizationCostModel  to take RTChecks.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10288
 
+  GeneratedRTChecks RTChecks(PSE, LVL, DT, LI, L,
+                             F->getParent()->getDataLayout());
----------------
fhahn wrote:
> does this need to be moved?
Same as above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109443