[PATCH] D52797: Clear LoopToAliasSetMap when outermost loop is deleted
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 2 15:24:19 PDT 2018
efriedma added a comment.
The commit message should refer to https://bugs.llvm.org/show_bug.cgi?id=31141 (assuming this patch fixes that bug).
================
Comment at: lib/Transforms/Scalar/LICM.cpp:1600
+ if (!L->getParentLoop())
+ LICM.getLoopToAliasSetMap().clear();
+
----------------
Need an explanation here for why we're clearing the map in exactly these circumstances. It isn't obvious that we won't lose useful cached information, or that we don't need to clear the map if the loop has a parent.
Repository:
rL LLVM
https://reviews.llvm.org/D52797
More information about the llvm-commits
mailing list