jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land.
> 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. That being said, the patch makes sense to me and it is a simple, straightforward improvement. I don't see any downsides and it simplifies the code. LGTM Repository: rC Clang https://reviews.llvm.org/D52118 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
