[PATCH] D44676: [SCEV] Make exact taken count calculation more optimistic

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 20 04:31:28 PDT 2018


fhahn added inline comments.


================
Comment at: lib/Analysis/ScalarEvolution.cpp:6672
+  const BasicBlock *Latch = L->getLoopLatch();
+  // All exits we have collected must dominate the only latch.
+  if (!Latch)
----------------
With exits you mean "exiting blocks", rather than "exit blocks", right? (http://lists.llvm.org/pipermail/llvm-dev/2017-July/115962.html)

 Maybe it would be clearer to explicitly use "exiting blocks" instead of exits here (and throughout the patch) :)



https://reviews.llvm.org/D44676





More information about the llvm-commits mailing list