made some changes based on review

Author: jiahanxie353

llvm/lib/Target/RISCV/GISel/RISCVRegisterBankInfo.cpp

@@ -314,16 +314,13 @@ RISCVRegisterBankInfo::getInstrMapping(const MachineInstr &MI) const {
     if (Ty.isVector()) {
       OpdsMapping[0] = getVRBValueMapping(Size.getKnownMinValue());
       OpdsMapping[1] = getVRBValueMapping(Size.getKnownMinValue());
-    } else if (isPreISelGenericFloatingPointOpcode(Opc)) {
-      OpdsMapping[0] = getFPValueMapping(Size.getFixedValue());
-      OpdsMapping[1] = getFPValueMapping(Size.getFixedValue());
     } else {
       OpdsMapping[0] = GPRValueMapping;
       OpdsMapping[1] = GPRValueMapping;
     }
     // Use FPR64 for s64 loads on rv32.
-    if (GPRSize == 32 && Ty.getSizeInBits().getKnownMinValue() == 64 &&
-        !Ty.isVector()) {
+    if (!Ty.isVector() && GPRSize == 32 &&
+        Ty.getSizeInBits().getKnownMinValue() == 64) {
       assert(MF.getSubtarget<RISCVSubtarget>().hasStdExtD());
       OpdsMapping[0] = getFPValueMapping(Ty.getSizeInBits());
       break;
@@ -347,17 +344,14 @@ RISCVRegisterBankInfo::getInstrMapping(const MachineInstr &MI) const {
     if (Ty.isVector()) {
       OpdsMapping[0] = getVRBValueMapping(Size.getKnownMinValue());
       OpdsMapping[1] = getVRBValueMapping(Size.getKnownMinValue());
-    } else if (isPreISelGenericFloatingPointOpcode(Opc)) {
-      OpdsMapping[0] = getFPValueMapping(Size.getFixedValue());
-      OpdsMapping[1] = getFPValueMapping(Size.getFixedValue());
     } else {
       OpdsMapping[0] = GPRValueMapping;
       OpdsMapping[1] = GPRValueMapping;
     }
 
     // Use FPR64 for s64 stores on rv32.
-    if (GPRSize == 32 && Ty.getSizeInBits().getKnownMinValue() == 64 &&
-        !Ty.isVector()) {
+    if (!Ty.isVector() && GPRSize == 32 &&
+        Ty.getSizeInBits().getKnownMinValue() == 64) {
       assert(MF.getSubtarget<RISCVSubtarget>().hasStdExtD());
       OpdsMapping[0] = getFPValueMapping(Ty.getSizeInBits());
       break;