[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

[Erich Keane via Phabricator via cfe-commits]

erichkeane added inline comments.


================
Comment at: clang/lib/Parse/ParseTemplate.cpp:923
+      // Skip until the semi-colon or a '}'.
+      SkipUntil(tok::r_brace, StopAtSemi | StopBeforeMatch);
+      TryConsumeToken(tok::semi);
----------------
aaron.ballman wrote:
> erichkeane wrote:
> > I'm not positive about this skip-until and 'TryConsumeToken' either.  The other place we do optional requires clause parsing does this, but the above 'ParseTemplateParameters' does not.
> I think we should be skipping until we hit a `>` or `>>` rather than ` }` right? We can recover after we've finished the template template parameter parsing, we don't need to parse to the end of the class.
Perhaps we should copy line 1000 and skip until `>`, `>>` or `,` with no consume?  


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

https://reviews.llvm.org/D110641