[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