royjacobson accepted this revision. royjacobson added a comment. This revision is now accepted and ready to land.
Code and added/modified tests LGTM! Do you think we should add a release note, given that it could break existing code? Its seems a bit unlikely, but the amount of broken tests have made me a bit worried. Also, maybe mention the github issue (https://github.com/llvm/llvm-project/issues/51173) in the commit message. ================ Comment at: clang/test/Parser/cxx-concepts-requires-clause.cpp:145 + template<typename T> struct B { ---------------- CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125711/new/ https://reviews.llvm.org/D125711 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
