[all-commits] [llvm/llvm-project] 22d65d: AMDGPU: Teach isOperandLegal about SALU literal re...

[Matt Arsenault via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 22d65d898961e96f0a8340e090ffa34558279eab
      https://github.com/llvm/llvm-project/commit/22d65d898961e96f0a8340e090ffa34558279eab
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-02-19 (Wed, 19 Feb 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
    M llvm/test/CodeGen/AMDGPU/fold-operands-scalar-fmac.mir
    M llvm/test/CodeGen/AMDGPU/fold-sgpr-multi-imm.mir

  Log Message:
  -----------
  AMDGPU: Teach isOperandLegal about SALU literal restrictions (#127626)

isOperandLegal mostly implemented the VALU operand rules, and
largely ignored SALU restrictions. This theoretically avoids
folding literals into SALU insts which already have a literal
operand. This issue is currently avoided due to a bug in
SIFoldOperands; this change will allow using raw operand
legality rules.

This breaks the formation of s_fmaak_f32 in SIFoldOperands,
but it probably should not have been forming there in the first
place. TwoAddressInsts or RA should generally handle that,
and this only worked by accident.



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