[PATCH] D93734: [LoopDeletion] Insert an early exit from dead path in loop

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopDeletion.cpp:110
   for (auto &I : L->blocks())
-    if (any_of(*I, [](Instruction &I) {
-          return I.mayHaveSideEffects() && !I.isDroppable();
-        }))
+    if (BBHasSideEffects(I))
       return false;
----------------
DriveBy:

This hurts. I know it was there before but `auto &I` is `BasicBlock *`... argh.
Could we please change that if we commit this to a `auto *` at least with a sensible variable name.


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

https://reviews.llvm.org/D93734