erichkeane wrote:
> Looks like there is a test
> (`clang-tools-extra/test/clang-tidy/infrastructure/diagnostic.cpp`) that
> contains the following:
>
> ```c++
> #ifdef PR64602 // Should not crash
> template <class T = void>
> struct S
> {
> auto foo(auto);
> };
>
> template <>
> auto S<>::foo(auto)
> {
> return 1;
> }
> // CHECK8: error: template parameter list matching the non-templated nested
> type 'S<>' should be empty ('template<>') [clang-diagnostic-error]
> #endif
> ```
>
> Which is failing because we now (correctly) accept this. Can I open a PR that
> addresses this single test, or would I need to revert and reapply the patch
> @erichkeane ?
If you act quickly, you can just do a new PR and get quick review from the code
owners on clang-tidy, which would not require a revert.
https://github.com/llvm/llvm-project/pull/80864
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
