From 8ccdbd19c89e9abc7d55e6b439e8f05561771053 Mon Sep 17 00:00:00 2001 From: Jim Grosbach Date: Tue, 26 Jun 2012 22:58:01 +0000 Subject: [PATCH] TableGen: AsmMatcher diagnostics preference detail. Don't override a custom diagnostic w/ a generic InvalidOperand, all else being equal. llvm-svn: 159238 --- llvm/utils/TableGen/AsmMatcherEmitter.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/llvm/utils/TableGen/AsmMatcherEmitter.cpp b/llvm/utils/TableGen/AsmMatcherEmitter.cpp index fb8d7048143b..f5e094e486a8 100644 --- a/llvm/utils/TableGen/AsmMatcherEmitter.cpp +++ b/llvm/utils/TableGen/AsmMatcherEmitter.cpp @@ -2663,7 +2663,9 @@ void AsmMatcherEmitter::run(raw_ostream &OS) { OS << " if (!HadMatchOtherThanPredicate &&\n"; OS << " (it == MnemonicRange.first || ErrorInfo <= i+1)) {\n"; OS << " ErrorInfo = i+1;\n"; - OS << " RetCode = Diag;\n"; + OS << " // InvalidOperand is the default. Prefer specificity.\n"; + OS << " if (Diag != Match_InvalidOperand)\n"; + OS << " RetCode = Diag;\n"; OS << " }\n"; OS << " // Otherwise, just reject this instance of the mnemonic.\n"; OS << " OperandsValid = false;\n";