[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.
More information about the llvm-commits