[PATCH] D11892: LoopVersioning: Use default LAA runtimeCheck, When input check not provided.

[Adam Nemet via llvm-commits]

anemet added inline comments.

================
Comment at: include/llvm/Transforms/Utils/LoopVersioning.h:33-37
@@ -32,7 +32,7 @@
 class LoopVersioning {
 public:
-  LoopVersioning(SmallVector<RuntimePointerChecking::PointerCheck, 4> Checks,
-                 const LoopAccessInfo &LAI, Loop *L, LoopInfo *LI,
-                 DominatorTree *DT,
-                 const SmallVector<int, 8> *PtrToPartition = nullptr);
+  LoopVersioning(
+      const LoopAccessInfo &LAI, Loop *L, LoopInfo *LI, DominatorTree *DT,
+      SmallVector<RuntimePointerChecking::PointerCheck, 4> *Checks = nullptr,
+      const SmallVector<int, 8> *PtrToPartition = nullptr);
 
----------------
How about adding another constructor that does not take Checks instead?  Then you should be able to initialize Checks with LAI.getRuntimePointerChecking()->getChecks().

I like the fact that LoopVersioning owns the checks.  That keeps things simple.


Repository:
  rL LLVM

http://reviews.llvm.org/D11892