diff options
author | Chris Lattner <sabre@nondot.org> | 2005-09-14 21:13:50 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2005-09-14 21:13:50 +0000 |
commit | 3ced3f8b82822571657b00d5e2701dccbe2015cf (patch) | |
tree | edd4b2fdf54e83d153450fe0bd9b9f6a53771b06 /llvm | |
parent | 9e4a4ee3dc3fd9f76cf2e627ac1bd34d904dd43f (diff) | |
download | bcm5719-llvm-3ced3f8b82822571657b00d5e2701dccbe2015cf.tar.gz bcm5719-llvm-3ced3f8b82822571657b00d5e2701dccbe2015cf.zip |
force all instruction operands to be named.
llvm-svn: 23358
Diffstat (limited to 'llvm')
-rw-r--r-- | llvm/utils/TableGen/CodeGenTarget.cpp | 7 | ||||
-rw-r--r-- | llvm/utils/TableGen/DAGISelEmitter.cpp | 3 |
2 files changed, 5 insertions, 5 deletions
diff --git a/llvm/utils/TableGen/CodeGenTarget.cpp b/llvm/utils/TableGen/CodeGenTarget.cpp index c18dff351f1..f0fdeae9456 100644 --- a/llvm/utils/TableGen/CodeGenTarget.cpp +++ b/llvm/utils/TableGen/CodeGenTarget.cpp @@ -285,8 +285,11 @@ CodeGenInstruction::CodeGenInstruction(Record *R, const std::string &AsmStr) throw "Unknown operand class '" + Rec->getName() + "' in instruction '" + R->getName() + "' instruction!"; - if (!DI->getArgName(i).empty() && - !OperandNames.insert(DI->getArgName(i)).second) + // Check that the operand has a name and that it's unique. + if (DI->getArgName(i).empty()) + throw "In instruction '" + R->getName() + "', operand #" + utostr(i) + + " has no name!"; + if (!OperandNames.insert(DI->getArgName(i)).second) throw "In instruction '" + R->getName() + "', operand #" + utostr(i) + " has the same name as a previous operand!"; diff --git a/llvm/utils/TableGen/DAGISelEmitter.cpp b/llvm/utils/TableGen/DAGISelEmitter.cpp index 0093e50f821..085f7d2dae5 100644 --- a/llvm/utils/TableGen/DAGISelEmitter.cpp +++ b/llvm/utils/TableGen/DAGISelEmitter.cpp @@ -788,10 +788,7 @@ void DAGISelEmitter::ParseAndResolveInstructions() { if (i == CGI.OperandList.size()) I->error("'" + InstResults.begin()->first + "' set but does not appear in operand list!"); - const std::string &OpName = CGI.OperandList[i].Name; - if (OpName.empty()) - I->error("Operand #" + utostr(i) + " in operands list has no name!"); // Check that it exists in InstResults. Record *R = InstResults[OpName]; |