[PATCH] D43177: Mark MergedLoadStoreMotion as not preserving MemDep results
Gerolf Hoflehner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 22 10:34:39 PST 2018
Gerolf added a comment.
Yes, this LGTM. I could not find a way to perform updates properly in the example. If recomputing the dependencies results in a compile-time issue we will have to dig deeper into MemDep.
Thanks for your fix + patience!
Cheers
Gerolf
Repository:
rL LLVM
https://reviews.llvm.org/D43177
More information about the llvm-commits
mailing list