[PATCH] D38374: [memcpyopt] Memcpy-memcpy dependence isn't detected across basic blocks

[Dan Gohman via Phabricator via llvm-commits]

sunfish added a comment.

Your suggestions above both involve adding functionality under a flag/option. That alone makes them different situations from just landing the current patch, in terms of seeing the work all the way through into production.

I have a simple, small, focused, incremental patch. It makes a significant improvement for some use cases. I have addressed all the concerns raised here about the patch itself, and am happy to address any others that are raised. The patch doesn't make anything about the MemorySSA-based patches harder to work on.


Repository:
  rL LLVM

https://reviews.llvm.org/D38374