summaryrefslogtreecommitdiffstats
path: root/llvm/utils/TableGen
diff options
context:
space:
mode:
authorHal Finkel <hfinkel@anl.gov>2014-03-22 11:33:32 +0000
committerHal Finkel <hfinkel@anl.gov>2014-03-22 11:33:32 +0000
commit68e03bd41f7f862cda7ebefd35d443fee1fd519f (patch)
tree16d37caf16578895b06f6ec2d2785e7fafc58fab /llvm/utils/TableGen
parent6690854109cc2d0626d62c9d7a91cfecf2130c9b (diff)
downloadbcm5719-llvm-68e03bd41f7f862cda7ebefd35d443fee1fd519f.tar.gz
bcm5719-llvm-68e03bd41f7f862cda7ebefd35d443fee1fd519f.zip
[TableGen] Don't assert, produce an error, when an instruction has too few operands
When an instruction's operand list does not have a sufficient number of operands to match with all of the variables that contribute to its encoding, instead of asserting inside a call to getSubOperandNumber, produce an informative error. llvm-svn: 204542
Diffstat (limited to 'llvm/utils/TableGen')
-rw-r--r--llvm/utils/TableGen/CodeEmitterGen.cpp13
1 files changed, 12 insertions, 1 deletions
diff --git a/llvm/utils/TableGen/CodeEmitterGen.cpp b/llvm/utils/TableGen/CodeEmitterGen.cpp
index a7fca06cf0f..c28b97e0420 100644
--- a/llvm/utils/TableGen/CodeEmitterGen.cpp
+++ b/llvm/utils/TableGen/CodeEmitterGen.cpp
@@ -107,9 +107,20 @@ AddCodeToMergeInOperand(Record *R, BitsInit *BI, const std::string &VarName,
while (NumberedOp < NumberOps &&
(CGI.Operands.isFlatOperandNotEmitted(NumberedOp) ||
(NamedOpIndices.size() && NamedOpIndices.count(
- CGI.Operands.getSubOperandNumber(NumberedOp).first))))
+ CGI.Operands.getSubOperandNumber(NumberedOp).first)))) {
++NumberedOp;
+ if (NumberedOp >= CGI.Operands.back().MIOperandNo +
+ CGI.Operands.back().MINumOperands) {
+ errs() << "Too few operands in record " << R->getName() <<
+ " (no match for variable " << VarName << "):\n";
+ errs() << *R;
+ errs() << '\n';
+
+ return;
+ }
+ }
+
OpIdx = NumberedOp++;
}
OpenPOWER on IntegriCloud