summaryrefslogtreecommitdiffstats
path: root/clang/lib/ASTMatchers/Dynamic/Parser.cpp
diff options
context:
space:
mode:
authorStephen Kelly <steveire@gmail.com>2019-12-29 14:51:22 +0000
committerStephen Kelly <steveire@gmail.com>2019-12-29 14:58:56 +0000
commitbee973f4df39f3eaf7fd91f4c3df6d0e64e9ee62 (patch)
tree4e2a5e69a7705b559d39fcbb0a496147cf927791 /clang/lib/ASTMatchers/Dynamic/Parser.cpp
parent544f200c785f0314949ba3b8d1c51f65bf8d7761 (diff)
downloadbcm5719-llvm-bee973f4df39f3eaf7fd91f4c3df6d0e64e9ee62.tar.gz
bcm5719-llvm-bee973f4df39f3eaf7fd91f4c3df6d0e64e9ee62.zip
Fix use of named values surrounded by newlines in clang-query
Diffstat (limited to 'clang/lib/ASTMatchers/Dynamic/Parser.cpp')
-rw-r--r--clang/lib/ASTMatchers/Dynamic/Parser.cpp15
1 files changed, 8 insertions, 7 deletions
diff --git a/clang/lib/ASTMatchers/Dynamic/Parser.cpp b/clang/lib/ASTMatchers/Dynamic/Parser.cpp
index ef209d1274a..061da8b8600 100644
--- a/clang/lib/ASTMatchers/Dynamic/Parser.cpp
+++ b/clang/lib/ASTMatchers/Dynamic/Parser.cpp
@@ -354,13 +354,6 @@ struct Parser::ScopedContextEntry {
bool Parser::parseIdentifierPrefixImpl(VariantValue *Value) {
const TokenInfo NameToken = Tokenizer->consumeNextToken();
- if (Tokenizer->nextTokenKind() == TokenInfo::TK_NewLine) {
- Error->addError(Tokenizer->peekNextToken().Range,
- Error->ET_ParserNoOpenParen)
- << "NewLine";
- return false;
- }
-
if (Tokenizer->nextTokenKind() != TokenInfo::TK_OpenParen) {
// Parse as a named value.
if (const VariantValue NamedValue =
@@ -388,6 +381,14 @@ bool Parser::parseIdentifierPrefixImpl(VariantValue *Value) {
}
return false;
}
+
+ if (Tokenizer->nextTokenKind() == TokenInfo::TK_NewLine) {
+ Error->addError(Tokenizer->peekNextToken().Range,
+ Error->ET_ParserNoOpenParen)
+ << "NewLine";
+ return false;
+ }
+
// If the syntax is correct and the name is not a matcher either, report
// unknown named value.
if ((Tokenizer->nextTokenKind() == TokenInfo::TK_Comma ||
OpenPOWER on IntegriCloud