[PATCH] D134128: [P0857R0 Part B] Resubmit an implemention for constrained template template parameters

[H. Vetinari via Phabricator via cfe-commits]

h-vetinari added inline comments.


================
Comment at: clang/docs/ReleaseNotes.rst:554
+- Implemented `P0857R0 <https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2017/p0857r0.html>`_,
+  which words for constrained lambdas and constrained template *template-parameter*\s.
 
----------------
erichkeane wrote:
> erichkeane wrote:
> > I'm not sure what you're trying to say here?  Is there just a typo that I'm missing that makes this perfectly clear? 
> Sorry, I'm still getting caught up on what 'which words' means here?  
Maybe the intention was something along the lines of: `words` -> `specifies` 


================
Comment at: clang/test/SemaTemplate/concepts.cpp:62
 
   // FIXME: This is valid under P0857R0.
   template<typename T> concept C = true;
----------------
That `FIXME` should be removed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134128