commit c798cc42831150f678b080c0994c1cc024bae653
author Devang Patel <dpatel@apple.com> Fri Jan 13 19:12:18 2012 +0000
committer Devang Patel <dpatel@apple.com> Fri Jan 13 19:12:18 2012 +0000
tree 50e11748954e0ff0d8b1c42d526b10c4ad8243f8
parent 4a5c0fd70e7a2001b682c8972dab6b0127313c8f

Refactor.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@148131 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/AsmParser/X86AsmParser.cpp

diff --git a/lib/Target/X86/AsmParser/X86AsmParser.cpp b/lib/Target/X86/AsmParser/X86AsmParser.cpp
index ce3831f..0a8f00d 100644
--- a/lib/Target/X86/AsmParser/X86AsmParser.cpp
+++ b/lib/Target/X86/AsmParser/X86AsmParser.cpp
@@ -48,6 +48,7 @@
   X86Operand *ParseOperand();
   X86Operand *ParseATTOperand();
   X86Operand *ParseIntelOperand();
+  X86Operand *ParseIntelMemOperand(StringRef TokenString, unsigned Size);
   X86Operand *ParseMemOperand(unsigned SegReg, SMLoc StartLoc);
 
   bool ParseDirectiveWord(unsigned Size, SMLoc L);
@@ -573,58 +574,63 @@
   return Size != 0;
 }
 
+/// ParseIntelMemOperand - Parse intel style memory operand.
+X86Operand *X86AsmParser::ParseIntelMemOperand(StringRef TokenString, 
+                                               unsigned Size) {
+  SMLoc Start = Parser.getTok().getLoc(), End;
+  unsigned SegReg = 0, BaseReg = 0, IndexReg = 0, Scale = 1;
+  Parser.Lex();
+  if (TokenString == "PTR")
+    Parser.Lex();
+    else {
+      Error(Start, "unexpected token!");
+      return 0;
+    }
+  
+  if (TokenString == "[")
+    Parser.Lex();
+  else {
+    Error(Start, "unexpected token!");
+    return 0;
+  }
+  
+  SMLoc LParenLoc = Parser.getTok().getLoc();
+  BaseReg = getIntelRegisterOperand(TokenString);
+  if (BaseReg == 0) {
+    Error(LParenLoc, "unexpected token!");
+    return 0;
+  }
+  Parser.Lex();
+  const MCExpr *Disp = MCConstantExpr::Create(0, getParser().getContext());
+  SMLoc ExprEnd;
+  if (getParser().ParseExpression(Disp, ExprEnd)) return 0;
+  End = Parser.getTok().getLoc();
+  if (TokenString == "]")
+    Parser.Lex();
+  if (BaseReg == 0) {
+    Error(End, "unexpected token!");
+    return 0;
+  }
+  return X86Operand::CreateMem(SegReg, Disp, BaseReg, IndexReg, Scale,
+			       Start, End, Size);
+}
+
 X86Operand *X86AsmParser::ParseIntelOperand() {
 
-  const AsmToken &Tok = Parser.getTok();
+  StringRef TokenString = Parser.getTok().getString();
   SMLoc Start = Parser.getTok().getLoc(), End;
 
   // register
-  if(unsigned RegNo = getIntelRegisterOperand(Tok.getString())) {
+  if(unsigned RegNo = getIntelRegisterOperand(TokenString)) {
     Parser.Lex();
     End = Parser.getTok().getLoc();
     return X86Operand::CreateReg(RegNo, Start, End);
   }
 
   // mem operand
-  unsigned SegReg = 0, BaseReg = 0, IndexReg = 0, Scale = 1;
-  StringRef OpStr = Tok.getString();
   unsigned Size = 0;
-  if (isIntelMemOperand(OpStr, Size)) {
-    Parser.Lex();
-    if (Tok.getString() == "PTR")
-      Parser.Lex();
-    else {
-      Error(Start, "unexpected token!");
-      return 0;
-    }
-
-    if (Tok.getString() == "[")
-      Parser.Lex();
-    else {
-      Error(Start, "unexpected token!");
-      return 0;
-    }
-
-    SMLoc LParenLoc = Parser.getTok().getLoc();
-    BaseReg = getIntelRegisterOperand(Tok.getString());
-    if (BaseReg == 0) {
-      Error(LParenLoc, "unexpected token!");
-      return 0;
-    }
-    Parser.Lex();
-    const MCExpr *Disp = MCConstantExpr::Create(0, getParser().getContext());
-    SMLoc ExprEnd;
-    if (getParser().ParseExpression(Disp, ExprEnd)) return 0;
-    End = Parser.getTok().getLoc();
-    if (Tok.getString() == "]")
-      Parser.Lex();
-    if (BaseReg == 0) {
-      Error(End, "unexpected token!");
-      return 0;
-    }
-    return X86Operand::CreateMem(SegReg, Disp, BaseReg, IndexReg, Scale,
-                                 Start, End, Size);
-  }
+  if (isIntelMemOperand(TokenString, Size))
+    ParseIntelMemOperand(TokenString, Size);
 
   // immediate.
   const MCExpr *Val;