[PATCH] D115590: [LoopNest] Add new utilites

[Whitney Tsang via Phabricator via llvm-commits]

Whitney marked an inline comment as done.
Whitney added inline comments.


================
Comment at: llvm/include/llvm/Analysis/LoopNestAnalysis.h:120
+  /// Get the loops in the nest at the given \p Depth.
+  LoopVectorTy getLoopsAtDepth(unsigned Depth) const {
+    assert(Depth >= Loops.front()->getLoopDepth() &&
----------------
Meinersbur wrote:
> Isn't it sufficient to return `Loops[Depth - getOutermostLoop().getLoopDepth()]`?
When the loop nest is not perfect, there can be multiple loops at the same depth.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115590