[PATCH] D40802: [MemDep] Don't use cached results from a previous larger query

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 17:38:19 PST 2017


sunfish marked an inline comment as done.
sunfish added inline comments.


================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:495
+    // Create the memset after removing the stores, so that if there any cached
+    // non-local dependencies on the removed instructions, the cache entries
+    // are updated to "dirty" entries pointing below the memset, so subsequent
----------------
hfinkel wrote:
> Please include the fact that you're talking about MemDep here, such as:
> 
> ... on the removed instructions in MemoryDependenceAnalysis, the...
Done.


Repository:
  rL LLVM

https://reviews.llvm.org/D40802





More information about the llvm-commits mailing list