massberg added inline comments.
================ Comment at: clang/unittests/AST/SourceLocationTest.cpp:274 + + Verifier.expectRange(2, 11, 2, 31); + EXPECT_TRUE(Verifier.match("template <typename T> concept Fooable = true;\n" ---------------- massberg wrote: > This range looks a bit to long. What I'm doing wrong? Arg, ok `T` is also a constrained template type parameter, but with this patch we consider the whole template type parameter as concept which is in this case wrong (but works for the constrained `auto` as function parameter). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155460/new/ https://reviews.llvm.org/D155460 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits