[llvm] 886b2b2 - [BPF] Use MCRegister. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 29 13:33:35 PDT 2024


Author: Craig Topper
Date: 2024-09-29T13:22:11-07:00
New Revision: 886b2b258f182f2e65f117f800f8d398b0f483c3

URL: https://github.com/llvm/llvm-project/commit/886b2b258f182f2e65f117f800f8d398b0f483c3
DIFF: https://github.com/llvm/llvm-project/commit/886b2b258f182f2e65f117f800f8d398b0f483c3.diff

LOG: [BPF] Use MCRegister. NFC

Added: 
    

Modified: 
    llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp b/llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp
index 06b7743e0cd310..32ddf11ec31968 100644
--- a/llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp
+++ b/llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp
@@ -86,7 +86,7 @@ struct BPFOperand : public MCParsedAsmOperand {
   } Kind;
 
   struct RegOp {
-    unsigned RegNum;
+    MCRegister RegNum;
   };
 
   struct ImmOp {
@@ -206,10 +206,10 @@ struct BPFOperand : public MCParsedAsmOperand {
     return Op;
   }
 
-  static std::unique_ptr<BPFOperand> createReg(unsigned RegNo, SMLoc S,
+  static std::unique_ptr<BPFOperand> createReg(MCRegister Reg, SMLoc S,
                                                SMLoc E) {
     auto Op = std::make_unique<BPFOperand>(Register);
-    Op->Reg.RegNum = RegNo;
+    Op->Reg.RegNum = Reg;
     Op->StartLoc = S;
     Op->EndLoc = E;
     return Op;
@@ -447,13 +447,13 @@ ParseStatus BPFAsmParser::parseRegister(OperandVector &Operands) {
     return ParseStatus::NoMatch;
   case AsmToken::Identifier:
     StringRef Name = getLexer().getTok().getIdentifier();
-    unsigned RegNo = MatchRegisterName(Name);
+    MCRegister Reg = MatchRegisterName(Name);
 
-    if (RegNo == 0)
+    if (!Reg)
       return ParseStatus::NoMatch;
 
     getLexer().Lex();
-    Operands.push_back(BPFOperand::createReg(RegNo, S, E));
+    Operands.push_back(BPFOperand::createReg(Reg, S, E));
   }
   return ParseStatus::Success;
 }
@@ -487,12 +487,12 @@ ParseStatus BPFAsmParser::parseImmediate(OperandVector &Operands) {
 bool BPFAsmParser::parseInstruction(ParseInstructionInfo &Info, StringRef Name,
                                     SMLoc NameLoc, OperandVector &Operands) {
   // The first operand could be either register or actually an operator.
-  unsigned RegNo = MatchRegisterName(Name);
+  MCRegister Reg = MatchRegisterName(Name);
 
-  if (RegNo != 0) {
+  if (Reg) {
     SMLoc E = SMLoc::getFromPointer(NameLoc.getPointer() - 1);
-    Operands.push_back(BPFOperand::createReg(RegNo, NameLoc, E));
-  } else if (BPFOperand::isValidIdAtStart (Name))
+    Operands.push_back(BPFOperand::createReg(Reg, NameLoc, E));
+  } else if (BPFOperand::isValidIdAtStart(Name))
     Operands.push_back(BPFOperand::createToken(Name, NameLoc));
   else
     return Error(NameLoc, "invalid register/token name");


        


More information about the llvm-commits mailing list