[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