[PATCH] D89847: [SCEV] Drop cached ranges of AddRecs after flag update

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 04:01:48 PDT 2020


mkazantsev added inline comments.


================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:5360
+    UnsignedRanges.erase(AddRec);
+    SignedRanges.erase(AddRec);
+  }
----------------
nikic wrote:
> Could it happen that other ranges have been computed in the meantime, based on the now outdated addrec range?
Unfortunately yes. But tracking them would be expensive.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89847/new/

https://reviews.llvm.org/D89847



More information about the llvm-commits mailing list