[PATCH] D144711: [DAGCombiner] don't reuse the pointer info for merged store

[ChenZheng via Phabricator via llvm-commits]

shchenz created this revision.
shchenz added reviewers: arsenm, foad, pengfei, aeubanks, nemanjai.
shchenz added a project: LLVM.
Herald added subscribers: StephenFan, ecnelises, hiraditya.
Herald added a project: All.
shchenz requested review of this revision.
Herald added subscribers: llvm-commits, wdng.

The merged store touches memory for other underlying objects, so mapping
the merged store to the first underlying object is not correct. For example 
in https://github.com/llvm/llvm-project/issues/60744, the merged store is
not correctly analyzed as dependent with memory operations which are also
part of the merged store.

Fix https://github.com/llvm/llvm-project/issues/60744


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D144711

Files:
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/test/CodeGen/AArch64/merge-scoped-aa-store.ll
  llvm/test/CodeGen/PowerPC/const-splat-array-init.ll
  llvm/test/CodeGen/PowerPC/extract-and-store.ll
  llvm/test/CodeGen/X86/merge-store-dependency.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D144711.500109.patch
Type: text/x-patch
Size: 7325 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230224/e55f87ac/attachment-0001.bin>