erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land.
So I think this is the right approach here. Unfortunately my knowledge of the parser is a little wonky, so I don't feel comfortable JUST approving this right away(though it DOES look correct to me). I'd like to give other folks a day or two to take a look. I've added the lang-wg, so please give them a day or two to point out why I'm an idiot here before committing this. ================ Comment at: clang/lib/Parse/ParseTemplate.cpp:850 + TemplateParameterDepthRAII CurTemplateDepthTracker(TemplateParameterDepth); + CurTemplateDepthTracker.addDepth(1); DefaultArg = ---------------- Same below. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151342/new/ https://reviews.llvm.org/D151342 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits