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

baziotis added a comment. > If that happens, this Phabricator review does not close automatically. It has > to be closed manually using the "Add Action..." dropdown. Noted. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74801/new/ https://reviews.llvm.org/D74801

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

baziotis added a comment. I committed that (https://reviews.llvm.org/rG21390eab4c05e0ed7e7d13ada9e85f62b87ea484) but as it seems, I should have added the differential division in the commit message. I'll try the next commit to be better. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D7

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

baziotis added a comment. In D74801#1898660 , @lattner wrote: > I'd be happy to help fix that problem. Please take a look at the llvm > developer policy. :-) Oh thanks, I thought I had to submit way more patches. CHANGES SINCE LAST ACTION https://r

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

baziotis added a comment. In D74801#1897267 , @lattner wrote: > Seems fine to me. Thank you! Please note that I don't have commit access. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74801/new/ https://reviews.llvm.org/D74801 _

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

baziotis added a comment. Ping :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74801/new/ https://reviews.llvm.org/D74801 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

baziotis added a comment. In D74801#1882155 , @lebedev.ri wrote: > lg Great! Just a note: I don't have commit access. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74801/new/ https://reviews.llvm.org/D74801 _

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

baziotis updated this revision to Diff 245295. baziotis edited the summary of this revision. baziotis added a comment. Self-cycle to self-loop CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74801/new/ https://reviews.llvm.org/D74801 Files: clang-tools-extra/clang-tidy/misc/NoRecursion

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

baziotis marked an inline comment as done. baziotis 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 &&

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

baziotis marked an inline comment as done. baziotis added a comment. In D74801#1881693 , @lebedev.ri wrote: > While i certainly agree that it's kinda confusing, i'm not really sure this > is conceptually correct change. > https://en.wikipedia.org/wiki/Lo

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

baziotis added a subscriber: jdoerfert. baziotis added a comment. In D74801#1881679 , @efriedma wrote: > Seems fine. Great, thanks and to @jdoerfert! Any way I can find the correct reviewers next time? Repository: rG LLVM Github Monorepo CHANGES SIN

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

baziotis created this revision. baziotis added reviewers: dexonsmith, lattner, timshen, chandlerc. Herald added subscribers: llvm-commits, cfe-commits, arphaman, hiraditya. Herald added projects: clang, LLVM. Not a super important change but yesterday I had to look in this function and initially