[all-commits] [llvm/llvm-project] a8c630: [DAGCombiner] add testcase for issue #60744; NFC

[Chen Zheng via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a8c630ea5b799cfc6f1f9843b9ce6be0c840b0a5
      https://github.com/llvm/llvm-project/commit/a8c630ea5b799cfc6f1f9843b9ce6be0c840b0a5
  Author: Chen Zheng <czhengsz at cn.ibm.com>
  Date:   2023-03-07 (Tue, 07 Mar 2023)

  Changed paths:
    A llvm/test/CodeGen/X86/merge-store-dependency.ll

  Log Message:
  -----------
  [DAGCombiner] add testcase for issue #60744; NFC


  Commit: fc26ab36a20a63c5aef5c46e2b7815b5369025a0
      https://github.com/llvm/llvm-project/commit/fc26ab36a20a63c5aef5c46e2b7815b5369025a0
  Author: Chen Zheng <czhengsz at cn.ibm.com>
  Date:   2023-03-07 (Tue, 07 Mar 2023)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/AArch64/argument-blocks-array-of-struct.ll
    M llvm/test/CodeGen/X86/merge-store-dependency.ll

  Log Message:
  -----------
  [DAGCombiner] don't use the pointer info for widen store

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.

Fixes #60744

Reviewed By: foad

Differential Revision: https://reviews.llvm.org/D144711


Compare: https://github.com/llvm/llvm-project/compare/795aebf38831...fc26ab36a20a