[PATCH] D134128: Resubmit an implemention for constrained template template parameters [P0857R0 Part B]
Liming Liu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Oct 16 07:31:01 PDT 2022
lime added a comment.
I think I located the problem. In the line 1014 of the file `SemaTemplateInstantiateDecl.cpp`, the requires clause is not instantiated as the parameters of the template parameter list, and these parameters have been instantiated with a shallower depth. So that's the reason why the depths are not confirm between the `T` in `template <template <C T> class>` and the constraints on the instantiated parameters. I guess adjusting depths should be unnecessary in `IsAtLeastAsConstrained`, if the requires clause was correctly instantiated, so are changes related to `SemaConcept.cpp`. I will work on this later.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134128/new/
https://reviews.llvm.org/D134128
More information about the cfe-commits
mailing list