[all-commits] [llvm/llvm-project] a56fa1: [clang-tidy] Fix failing test after #80864 (#81171)

Krystian Stasiowski via All-commits all-commits at lists.llvm.org
Thu Feb 8 10:59:59 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a56fa161ab2617fa3aab3f91285fc757b6a8e09b
      https://github.com/llvm/llvm-project/commit/a56fa161ab2617fa3aab3f91285fc757b6a8e09b
  Author: Krystian Stasiowski <sdkrystian at gmail.com>
  Date:   2024-02-08 (Thu, 08 Feb 2024)

  Changed paths:
    M clang-tools-extra/test/clang-tidy/infrastructure/diagnostic.cpp

  Log Message:
  -----------
  [clang-tidy] Fix failing test after #80864 (#81171)

The following test case in
`clang-tools-extra/test/clang-tidy/infrastructure/diagnostic.cpp` is
failing:
```
#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
```

#80864 fixes a bug where we would (incorrectly) append invented template
parameters to empty template parameter lists, which causes this test to
fail.




More information about the All-commits mailing list