[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:45:01 PDT 2016


niravd requested changes to this revision.
niravd added a comment.

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