[PATCH] D132503: [clang] Add cxx scope if needed for requires clause.

Luke Nihlen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 09:41:12 PDT 2022


luken-google added inline comments.


================
Comment at: clang/lib/Parse/ParseTemplate.cpp:293
+    if (Tok.is(tok::kw_requires)) {
+      CXXScopeSpec ScopeSpec = DeclaratorInfo.getCXXScopeSpec();
+      DeclaratorScopeObj DeclScopeObj(*this, ScopeSpec);
----------------
erichkeane wrote:
> Not sure about this type, should this be a reference?
Mm, good point. Fixed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132503/new/

https://reviews.llvm.org/D132503



More information about the cfe-commits mailing list