[PATCH] D22816: Move assert as early as possible.

[Sebastian Pop via llvm-commits]

sebpop added inline comments.

================
Comment at: llvm/include/llvm/Analysis/LoopInfoImpl.h:249
@@ -248,3 +248,3 @@
     for (typename BlockTraits::ChildIteratorType SI =
            BlockTraits::child_begin(BB), SE = BlockTraits::child_end(BB);
          SI != SE; ++SI)
----------------
sanjoy wrote:
> Since you're touching this code, can you (in a separate change) change this loop to use `std::any_of`?   Even better would be if you add a `children` function (that directly returns a range) and use `llvm::any_of`.
Cleanup committed in https://reviews.llvm.org/rL276835


Repository:
  rL LLVM

https://reviews.llvm.org/D22816