[all-commits] [llvm/llvm-project] 9b1051: [DAG] Use SDValue for PatFrag checks (#137519)

[David Green via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9b1051281e439fcf6f6ccf03766c5bcf04ceec4b
      https://github.com/llvm/llvm-project/commit/9b1051281e439fcf6f6ccf03766c5bcf04ceec4b
  Author: David Green <david.green at arm.com>
  Date:   2025-05-01 (Thu, 01 May 2025)

  Changed paths:
    M llvm/include/llvm/CodeGen/SelectionDAGISel.h
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
    M llvm/lib/Target/AArch64/AArch64InstrInfo.td
    M llvm/lib/Target/AMDGPU/SIInstrInfo.td
    M llvm/lib/Target/AMDGPU/SIInstructions.td
    M llvm/lib/Target/ARM/ARMInstrInfo.td
    M llvm/lib/Target/RISCV/RISCVInstrInfo.td
    M llvm/lib/Target/WebAssembly/WebAssemblyInstrInfo.td
    M llvm/lib/Target/X86/X86InstrSSE.td
    M llvm/test/CodeGen/AArch64/aarch64-mull-masks.ll
    M llvm/test/TableGen/HasNoUse.td
    M llvm/test/TableGen/address-space-patfrags.td
    M llvm/utils/TableGen/Common/CodeGenDAGPatterns.cpp
    M llvm/utils/TableGen/DAGISelMatcherEmitter.cpp

  Log Message:
  -----------
  [DAG] Use SDValue for PatFrag checks (#137519)

If the SDNode is used it can pick up the wrong results number, for
example looking at the known bits of the first result where it should be
looking at the second. The SDValue is already present as the
SelectCodeCommon checks move from parent to child, pass the SDValue
through to CheckNodePredicate as Op so that it can use it if necessary.
SDNode *N is still generated, keeping most PatFrags the same.

Fixes #137274



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications