aaron.ballman added reviewers: erichkeane, clang-language-wg. aaron.ballman added a comment.
FWIW, I think this looks correct as well. I've added some reviewers just in case there's something I've missed regarding concepts. ================ Comment at: clang/lib/Parse/ParseTemplate.cpp:293-301 + DeclaratorScopeObj DeclScopeObj(*this, DeclaratorInfo.getCXXScopeSpec()); + if (DeclaratorInfo.getCXXScopeSpec().isValid()) { + if (Actions.ShouldEnterDeclaratorScope( + getCurScope(), DeclaratorInfo.getCXXScopeSpec())) { + DeclScopeObj.EnterDeclaratorScope(); + } + } ---------------- Can simplify even further with a new local. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132503/new/ https://reviews.llvm.org/D132503 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits