From ad61925e27e891e13f8576086cbbbc60a01f5727 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Fri, 30 Sep 2005 04:42:31 +0000 Subject: [PATCH] Generate a parse error instead of a checked exception if template args are used on a def. llvm-svn: 23545 --- llvm/utils/TableGen/FileParser.y | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/llvm/utils/TableGen/FileParser.y b/llvm/utils/TableGen/FileParser.y index 8fc704280f5e..9be816ecb4dc 100644 --- a/llvm/utils/TableGen/FileParser.y +++ b/llvm/utils/TableGen/FileParser.y @@ -559,15 +559,15 @@ DefName : ObjectName { Records.addDef(CurRec); }; -ObjectBody : OptTemplateArgList ClassList { +ObjectBody : ClassList { ParsingTemplateArgs = false; - for (unsigned i = 0, e = $2->size(); i != e; ++i) { - addSubClass((*$2)[i].first, *(*$2)[i].second); + for (unsigned i = 0, e = $1->size(); i != e; ++i) { + addSubClass((*$1)[i].first, *(*$1)[i].second); // Delete the template arg values for the class - delete (*$2)[i].second; + delete (*$1)[i].second; } - delete $2; // Delete the class list... - + delete $1; // Delete the class list... + // Process any variables on the set stack... for (unsigned i = 0, e = LetStack.size(); i != e; ++i) for (unsigned j = 0, e = LetStack[i].size(); j != e; ++j) @@ -579,19 +579,15 @@ ObjectBody : OptTemplateArgList ClassList { CurRec = 0; }; -ClassInst : CLASS ClassName ObjectBody { - $$ = $3; -}; +ClassInst : CLASS ClassName OptTemplateArgList ObjectBody { + $$ = $4; + }; DefInst : DEF DefName ObjectBody { $3->resolveReferences(); // If ObjectBody has template arguments, it's an error. - if (!$3->getTemplateArgs().empty()) { - err() << "Def '" << $3->getName() - << "' is not permitted to have template arguments!\n"; - exit(1); - } + assert($3->getTemplateArgs().empty() && "How'd this get template args?"); $$ = $3; };