[PATCH] D112294: [SCEV][NFC] Win some compile time from mass forgetMemoizedResults

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 22 10:10:04 PDT 2021


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

LGTM



================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:12863
 
+void ScalarEvolution::forgetMemoizedResultsImpl(const SCEV *S) {
+  ValuesAtScopes.erase(S);
----------------
Personally, I'd inline this into the single caller as having it out of line doesn't seem to add any value, but I don't hugely care either way.


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

https://reviews.llvm.org/D112294



More information about the llvm-commits mailing list