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

[Nicolai Hähnle via llvm-commits]

nhaehnle added a comment.

To follow up, these are the tests with asan failures (when https://reviews.llvm.org/D25313 is applied) that are fixed by applying this patch (AMDGPU and X86 backends only):

>   LLVM :: CodeGen/AMDGPU/amdgpu.private-memory.ll
>   LLVM :: CodeGen/AMDGPU/invariant-load-no-alias-store.ll
>   LLVM :: CodeGen/AMDGPU/merge-stores.ll
>   LLVM :: CodeGen/AMDGPU/vector-alloca.ll
>   LLVM :: CodeGen/X86/2012-11-28-merge-store-alias.ll
>   LLVM :: CodeGen/X86/2012-12-1-merge-multiple.ll
>   LLVM :: CodeGen/X86/MergeConsecutiveStores.ll
>   LLVM :: CodeGen/X86/avx-vextractf128.ll
>   LLVM :: CodeGen/X86/merge_store.ll
>   LLVM :: CodeGen/X86/pr18023.ll
>   LLVM :: CodeGen/X86/stores-merging.ll
>   LLVM :: CodeGen/X86/vector-merge-store-fp-constants.ll


https://reviews.llvm.org/D25601