[PATCH] D93665: [LoopNest] Allow empty basic blocks without loops

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 4 10:06:29 PST 2021


Meinersbur accepted this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Analysis/LoopNestAnalysis.cpp:223
+
+  SmallPtrSet<const BasicBlock *, 4> Visited;
+  const BasicBlock *BB = From->getSingleSuccessor();
----------------
Whitney wrote:
> Meinersbur wrote:
> > To clarify: the `Visited` set is used to detect an infinite loop?
> Yes, to avoid running into an infinite loop.
Could you add a comment to clarify?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93665



More information about the llvm-commits mailing list