[PATCH] D89149: [SelectionDAG] Fix alias checking with potential later stack reuse

Thu Oct 15 14:00:05 PDT 2020

yonghong-song added a comment.

In D89149#2333147 <https://reviews.llvm.org/D89149#2333147>, @niravd wrote:

> As an alternative, we could update ImproveChain (and visitLIFETIME_END) to limit the aliasing around lifetime_start / end to disallow improving the chain dependence of a mem op node from a different lifetime node that may alias. That should prohibit access from two aliasable frame indices from being concurrent while still allowing us to leverage that disjoint allocs should be disjoint.

I am not familiar with selection dag code base. Could you help draft a patch for this? I can help do some testing. Feel free to use my test case at https://reviews.llvm.org/D88525.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89149/new/

https://reviews.llvm.org/D89149