[llvm-branch-commits] [llvm] AMDGPU: Fix DPP combiner using isOperandLegal on incomplete inst (PR #155595)
Frederik Harwath via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Wed Aug 27 07:22:57 PDT 2025
================
@@ -250,7 +250,7 @@ MachineInstr *GCNDPPCombine::createDPPInst(MachineInstr &OrigMI,
++NumOperands;
}
if (auto *SDst = TII->getNamedOperand(OrigMI, AMDGPU::OpName::sdst)) {
- if (TII->isOperandLegal(*DPPInst.getInstr(), NumOperands, SDst)) {
+ if (AMDGPU::hasNamedOperand(DPPOp, AMDGPU::OpName::sdst)) {
----------------
frederik-h wrote:
The `TII->isOperandLegal` invocations for the `srcX` operands have been moved to a new loop below, but it seems that this one went away, i.e. it is not contained in the list of operands being checked in the loop. Is this intended?
https://github.com/llvm/llvm-project/pull/155595
More information about the llvm-branch-commits
mailing list