================
@@ -1063,7 +1063,8 @@ Parser::isCXXDeclarationSpecifier(ImplicitTypenameContext 
AllowImplicitTypename,
       return TPResult::False;
     }
 
-    if (Next.isNot(tok::coloncolon) && Next.isNot(tok::less)) {
+    if (Next.isNot(tok::coloncolon) && Next.isNot(tok::less) &&
----------------
AaronBallman wrote:

Hmm, we have `Token::isOneOf()` but don't have `Token::isNoneOf()`; I wonder if 
we should just add that implementation and use it here rather than grow to a 
third check of the token kind? (Maybe as an NFC follow-up which replaces serial 
`isNot()` checks in other places too?)

I don't insist, these changes are correct as-is.

https://github.com/llvm/llvm-project/pull/169246
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to