[PATCH] D25601: DAGCombiner: fix use-after-free when merging consecutive stores

Nirav Dave via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 12:44:57 PDT 2016


niravd requested changes to this revision.
niravd added a comment.
This revision now requires changes to proceed.

I'm still not convinced this fixes the problem. It reduces the number of merges that we do but they should should have been safe as is. .  Let's take some time to look at the failing test cases and see if we can find where the problem is coming from.


https://reviews.llvm.org/D25601





More information about the llvm-commits mailing list