[PATCH] D38435: [SCEV] Maintain and use a loop->loop invalidation dependency

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 13 02:20:14 PDT 2017


mkazantsev accepted this revision.
mkazantsev added a comment.
This revision is now accepted and ready to land.

LGTM with nits inlined.



================
Comment at: lib/Analysis/ScalarEvolution.cpp:6414
+        else {
+          BackedgeTakenCounts.erase(LoopOrSCEV.get<const Loop *>());
+          PredicatedBackedgeTakenCounts.erase(LoopOrSCEV.get<const Loop *>());
----------------
mkazantsev wrote:
> Why we are not using `forgetLoop` here?
Scratch this, I've re-written the commit message and understood the point. :)


https://reviews.llvm.org/D38435





More information about the llvm-commits mailing list