[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