[PATCH] D52118: [Loopinfo] Remove one latch case in getLoopID. NFC.

Michael Kruse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 17 11:43:58 PDT 2018


Meinersbur added a comment.

In https://reviews.llvm.org/D52118#1235897, @jdoerfert wrote:

> > save an iteration over the loop's basic blocks (which is what getLoopLatch does)
>
> I'm not sure this is true. getLoopLatch() in LoopInfoImpl.h 
>  only traverses the children of the header in the inverse graph.
>  That should, I think, be similar to predecessors(Header) in case
>  of the IR CFG.


You're right. I changed the summary/commit message accordingly.


Repository:
  rC Clang

https://reviews.llvm.org/D52118





More information about the cfe-commits mailing list