================
@@ -8097,8 +8095,16 @@ void SIInstrInfo::legalizeOperandsVALUt16(MachineInstr
&MI, unsigned OpIdx,
if (!RI.isVGPRClass(CurrRC))
return;
- int16_t RCID = getOpRegClassID(get(Opcode).operands()[OpIdx]);
- const TargetRegisterClass *ExpectedRC = RI.getRegClass(RCID);
+ const TargetRegisterClass *ExpectedRC;
+ if (MI.isPHI()) {
+ // A PHI is generic, so it carries no operand register classes.
+ ExpectedRC = MRI.getRegClass(MI.getOperand(0).getReg());
----------------
yxsamliu wrote:
Could this use `getOpRegClass(MI, 0)`? It already handles generic instructions
and is also used by the PHI legalization below. This would keep the
register-class lookup in one place.
https://github.com/llvm/llvm-project/pull/222155
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
