[PATCH] D18062: Fix Load Control Dependence in MemCpy Generation

[Hal Finkel via llvm-commits]

hfinkel added a comment.

In http://reviews.llvm.org/D18062#381709, @niravd wrote:

> Ping. Your thoughts on this Hal?


This is quite interesting. The question is: Can we guarantee that all data dependencies are also encoded as chain dependencies? If so, then I'm happy with this -- although I'd also like some loud comments (maybe in include/llvm/CodeGen/ISDOpcodes.h), and ideally some verification code which can be enabled under expensive checks.

My worry here is not just about what we generate, but about what FindBetterChain might do. It would need to specifically not remove a chain dependency that was present to protect/represent a value dependency.

> ppc64-align-long-double.ll now may see multiple serializations of its stores


What does this mean? I see you've put in CHECK-DAGs, but do you mean the order has changed or that there are now multiple stores?


http://reviews.llvm.org/D18062