[PATCH] D53847: [C++2a] P0634r3: Down with typename!
Alan Zhao via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 27 15:41:01 PDT 2022
ayzhao added a comment.
In D53847#3819200 <https://reviews.llvm.org/D53847#3819200>, @cor3ntin wrote:
> Good catch
>
> template<typename T>
> concept K = requires (typename T::Type X) { // (3)
> X.next();
> }
>
> That typename should be optional. Can you add a test for that? If it works you can mark P2092 <https://reviews.llvm.org/P2092> as fully supported.
Done.
The test originally failed, but it was a simple fix to get it to work.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53847/new/
https://reviews.llvm.org/D53847
More information about the cfe-commits
mailing list