[MCParser] Remove parseParenExprOfDepth

Introduced by http://reviews.llvm.org/D9742 as a hack, which then became
unneeded.

Primary test: llvm/test/MC/Mips/memory-offsets.s

Author: MaskRay

llvm/include/llvm/MC/MCParser/MCAsmParser.h

@@ -331,17 +331,6 @@ class MCAsmParser {
   /// \return - False on success.
   virtual bool checkForValidSection() = 0;
 
-  /// Parse an arbitrary expression of a specified parenthesis depth,
-  /// assuming that the initial '(' characters have already been consumed.
-  ///
-  /// \param ParenDepth - Specifies how many trailing expressions outside the
-  /// current parentheses we have to parse.
-  /// \param Res - The value of the expression. The result is undefined
-  /// on error.
-  /// \return - False on success.
-  virtual bool parseParenExprOfDepth(unsigned ParenDepth, const MCExpr *&Res,
-                                     SMLoc &EndLoc) = 0;
-
   /// Parse a .gnu_attribute.
   bool parseGNUAttribute(SMLoc L, int64_t &Tag, int64_t &IntegerValue);
 };

llvm/lib/MC/MCParser/AsmParser.cpp

@@ -272,8 +272,6 @@ class AsmParser : public MCAsmParser {
   bool parsePrimaryExpr(const MCExpr *&Res, SMLoc &EndLoc,
                         AsmTypeInfo *TypeInfo) override;
   bool parseParenExpression(const MCExpr *&Res, SMLoc &EndLoc) override;
-  bool parseParenExprOfDepth(unsigned ParenDepth, const MCExpr *&Res,
-                             SMLoc &EndLoc) override;
   bool parseAbsoluteExpression(int64_t &Res) override;
 
   /// Parse a floating point expression using the float \p Semantics
@@ -1536,26 +1534,6 @@ bool AsmParser::parseParenExpression(const MCExpr *&Res, SMLoc &EndLoc) {
   return parseParenExpr(Res, EndLoc) || parseBinOpRHS(1, Res, EndLoc);
 }
 
-bool AsmParser::parseParenExprOfDepth(unsigned ParenDepth, const MCExpr *&Res,
-                                      SMLoc &EndLoc) {
-  if (parseParenExpr(Res, EndLoc))
-    return true;
-
-  for (; ParenDepth > 0; --ParenDepth) {
-    if (parseBinOpRHS(1, Res, EndLoc))
-      return true;
-
-    // We don't Lex() the last RParen.
-    // This is the same behavior as parseParenExpression().
-    if (ParenDepth - 1 > 0) {
-      EndLoc = getTok().getEndLoc();
-      if (parseRParen())
-        return true;
-    }
-  }
-  return false;
-}
-
 bool AsmParser::parseAbsoluteExpression(int64_t &Res) {
   const MCExpr *Expr;
 

llvm/lib/MC/MCParser/MasmParser.cpp

@@ -547,8 +547,6 @@ class MasmParser : public MCAsmParser {
   bool parsePrimaryExpr(const MCExpr *&Res, SMLoc &EndLoc,
                         AsmTypeInfo *TypeInfo) override;
   bool parseParenExpression(const MCExpr *&Res, SMLoc &EndLoc) override;
-  bool parseParenExprOfDepth(unsigned ParenDepth, const MCExpr *&Res,
-                             SMLoc &EndLoc) override;
   bool parseAbsoluteExpression(int64_t &Res) override;
 
   /// Parse a floating point expression using the float \p Semantics
@@ -1905,26 +1903,6 @@ bool MasmParser::parseParenExpression(const MCExpr *&Res, SMLoc &EndLoc) {
   return parseParenExpr(Res, EndLoc) || parseBinOpRHS(1, Res, EndLoc);
 }
 
-bool MasmParser::parseParenExprOfDepth(unsigned ParenDepth, const MCExpr *&Res,
-                                       SMLoc &EndLoc) {
-  if (parseParenExpr(Res, EndLoc))
-    return true;
-
-  for (; ParenDepth > 0; --ParenDepth) {
-    if (parseBinOpRHS(1, Res, EndLoc))
-      return true;
-
-    // We don't Lex() the last RParen.
-    // This is the same behavior as parseParenExpression().
-    if (ParenDepth - 1 > 0) {
-      EndLoc = getTok().getEndLoc();
-      if (parseRParen())
-        return true;
-    }
-  }
-  return false;
-}
-
 bool MasmParser::parseAbsoluteExpression(int64_t &Res) {
   const MCExpr *Expr;
 

llvm/lib/Target/Mips/AsmParser/MipsAsmParser.cpp

@@ -6516,7 +6516,7 @@ bool MipsAsmParser::parseMemOffset(const MCExpr *&Res, bool isParenExpr) {
   SMLoc S;
 
   if (isParenExpr)
-    return getParser().parseParenExprOfDepth(0, Res, S);
+    return getParser().parseExpression(Res, S) || getParser().parseRParen();
   return getParser().parseExpression(Res);
 }