[llvm] [RISCV][GISel] Remove source constraint from selectCopy and use it fo… (PR #67207)

Michael Maitland via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 22 16:35:37 PDT 2023


================
@@ -271,37 +270,25 @@ const TargetRegisterClass *RISCVInstructionSelector::getRegClassForTypeOnBank(
 bool RISCVInstructionSelector::selectCopy(MachineInstr &MI,
                                           MachineRegisterInfo &MRI) const {
   Register DstReg = MI.getOperand(0).getReg();
-  Register SrcReg = MI.getOperand(1).getReg();
 
-  if (Register::isPhysicalRegister(SrcReg) &&
-      Register::isPhysicalRegister(DstReg))
+  if (DstReg.isPhysical())
----------------
michaelmaitland wrote:

Are the select functions supposed to return `true` if a change was made?

https://github.com/llvm/llvm-project/pull/67207


More information about the llvm-commits mailing list