[llvm] [AMDGPU] Handle subregisters properly in generic operand legalizer (PR #108496)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 12 23:02:13 PDT 2024
================
@@ -6231,10 +6231,11 @@ void SIInstrInfo::legalizeGenericOperand(MachineBasicBlock &InsertMBB,
return;
Register DstReg = MRI.createVirtualRegister(DstRC);
+ Op.setSubReg(0);
----------------
arsenm wrote:
Instead of clearing this and using .add, could use .addReg and just not take the subreg index
We probably don't want to be preserving the flags here. If this somehow ended up with a kill, it would be wrong
https://github.com/llvm/llvm-project/pull/108496
More information about the llvm-commits
mailing list