[PATCH] D74801: [ADT][NFC] SCCIterator: Change hasLoop() to hasCycle()

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 18 15:43:39 PST 2020


lebedev.ri added inline comments.


================
Comment at: llvm/tools/opt/PrintSCC.cpp:82-83
       errs() << (*I)->getName() << ", ";
-    if (nextSCC.size() == 1 && SCCI.hasLoop())
-      errs() << " (Has self-loop).";
+    if (nextSCC.size() == 1 && SCCI.hasCycle())
+      errs() << " (Has self-cycle).";
   }
----------------
baziotis wrote:
> lebedev.ri wrote:
> > err, i was specifically talking about this
> Maybe we can keep the "Has self-loop here".
Then this would look good to me


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74801





More information about the llvm-commits mailing list