[PATCH] D27469: [LoopVersioning] Require loop-simplify form for loop versioning.

[Ashutosh Nema via Phabricator via llvm-commits]

ashutosh.nema added a comment.

It’s good to avoid loop which are not in loop-simplify form.

This looks OK to me except a minor comment.



================
Comment at: lib/Transforms/Scalar/LoopVersioningLICM.cpp:454
+
+  if (!CurLoop->isLoopSimplifyForm()) {
+    DEBUG(
----------------
Please move this check to "legalLoopStructure".


https://reviews.llvm.org/D27469