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

Reply via email to