[all-commits] [llvm/llvm-project] 850bb8: [clang-tidy] RenamerClangTidy group redecls into 1...

[Nathan James via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 850bb889a56ccf3252792c6d3db59542e94753ae
      https://github.com/llvm/llvm-project/commit/850bb889a56ccf3252792c6d3db59542e94753ae
  Author: Nathan James <n.james93 at hotmail.co.uk>
  Date:   2020-06-18 (Thu, 18 Jun 2020)

  Changed paths:
    M clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming.cpp

  Log Message:
  -----------
  [clang-tidy] RenamerClangTidy group redecls into 1 warning.

This changes the behavious of `RenamerClangTidyCheck` based checks by grouping declarations of the same thing into 1 warning where it is first declared.
This cleans up clang-tidy output and prevents issues where 1 fix-it couldn't be applied, yet all other warnings(and fix-its) for the same declaration would be applied.
The old behaviour of forward declaring a class without defining it isn't affected, i.e. no warnings will be emitted for that case.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D82059