fix comments

Author: Shoreshen

llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp

@@ -4394,8 +4394,6 @@ enum class TypeClass { VECTOR_OF_TWO, SCALAR, NONE_OF_LISTED };
 
 static TypeClass isVectorOfTwoOrScalar(const MachineOperand *Op,
                                        const MachineRegisterInfo &MRI) {
-  if (!Op->isReg() || Op->getReg().isPhysical())
-    return TypeClass::NONE_OF_LISTED;
   LLT OpTy = MRI.getType(Op->getReg());
   if (OpTy.isScalar())
     return TypeClass::SCALAR;
@@ -4419,7 +4417,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrHi, CurrLo] = [-OpHi, -OpLo](2 x Type)
       // [SrcHi, SrcLo]   = [-OpHi, -OpLo]
       return SrcStatus::IS_BOTH_NEG;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // [SrcHi, SrcLo]   = [CurrHi, CurrLo]
       // [CurrHi, CurrLo] = neg [OpHi, OpLo](Type)
@@ -4436,7 +4435,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrHi, CurrLo] = [-OpHi, -OpLo](2 x Type)
       // [SrcHi, SrcLo]   = [-(-OpHi), -OpLo] = [OpHi, -OpLo]
       return SrcStatus::IS_LO_NEG;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // [SrcHi, SrcLo]   = [-CurrHi, CurrLo]
       // [CurrHi, CurrLo] = neg [OpHi, OpLo](Type)
@@ -4453,7 +4453,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrHi, CurrLo] = [-OpHi, -OpLo](2 x Type)
       // [SrcHi, SrcLo]   = [-OpHi, -(-OpLo)] = [-OpHi, OpLo]
       return SrcStatus::IS_HI_NEG;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // [SrcHi, SrcLo]   = [CurrHi, -CurrLo]
       // [CurrHi, CurrLo] = fneg [OpHi, OpLo](Type)
@@ -4470,7 +4471,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrHi, CurrLo] = [-OpHi, -OpLo](2 x Type)
       // [SrcHi, SrcLo]   = [OpHi, OpLo]
       return SrcStatus::IS_SAME;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // [SrcHi, SrcLo]   = [-CurrHi, -CurrLo]
       // [CurrHi, CurrLo] = fneg [OpHi, OpLo](Type)
@@ -4503,7 +4505,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrUpper, CurrLower] = [-OpUpper, -OpLower](2 x Type)
       // Src = -OpLower
       return SrcStatus::IS_LOWER_HALF_NEG;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // Src = CurrLower
       // Curr = [CurrUpper, CurrLower]
@@ -4537,7 +4540,8 @@ static SrcStatus getNegStatus(const MachineOperand *Op, SrcStatus S,
       // [CurrUpper, CurrLower] = [-OpUpper, -OpLower](2 x Type)
       // Src = -(-OpLower) = OpLower
       return SrcStatus::IS_LOWER_HALF;
-    } else if (NegType == TypeClass::SCALAR) {
+    }
+    if (NegType == TypeClass::SCALAR) {
       // Scalar:
       // Src = -CurrLower
       // Curr = [CurrUpper, CurrLower]
@@ -4566,9 +4570,10 @@ calcNextStatus(std::pair<const MachineOperand *, SrcStatus> Curr,
 
   // Handle general Opc cases.
   switch (Opc) {
-  case AMDGPU::G_BITCAST:
   case AMDGPU::G_CONSTANT:
   case AMDGPU::G_FCONSTANT:
+    return retOpStat(&MI->getOperand(1), Curr.second, Curr);
+  case AMDGPU::G_BITCAST:
   case AMDGPU::COPY:
     return retOpStat(&MI->getOperand(1), Curr.second, Curr);
   case AMDGPU::G_FNEG: