diff options
author | Stephen Kelly <steveire@gmail.com> | 2019-12-29 14:51:22 +0000 |
---|---|---|
committer | Stephen Kelly <steveire@gmail.com> | 2019-12-29 14:58:56 +0000 |
commit | bee973f4df39f3eaf7fd91f4c3df6d0e64e9ee62 (patch) | |
tree | 4e2a5e69a7705b559d39fcbb0a496147cf927791 /clang/lib/ASTMatchers/Dynamic/Parser.cpp | |
parent | 544f200c785f0314949ba3b8d1c51f65bf8d7761 (diff) | |
download | bcm5719-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.cpp | 15 |
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 || |