[all-commits] [llvm/llvm-project] ddebed: [clang-tidy] performance-* checks: Match AllowedTy...

[Felix Berger via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ddebed8e9742add4372d54021cb55e06b655cfd6
      https://github.com/llvm/llvm-project/commit/ddebed8e9742add4372d54021cb55e06b655cfd6
  Author: Felix Berger <flx at google.com>
  Date:   2021-04-06 (Tue, 06 Apr 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/utils/Matchers.h
    M clang-tools-extra/docs/clang-tidy/checks/performance-for-range-copy.rst
    M clang-tools-extra/docs/clang-tidy/checks/performance-unnecessary-copy-initialization.rst
    M clang-tools-extra/docs/clang-tidy/checks/performance-unnecessary-value-param.rst
    M clang-tools-extra/test/clang-tidy/checkers/performance-for-range-copy-allowed-types.cpp

  Log Message:
  -----------
  [clang-tidy] performance-* checks: Match AllowedTypes against qualified type names when they contain "::".

This allows users to be more precise and exclude a type in a specific namespace
from triggering the check instead of excluding all types with the same
unqualified name.

This change should not interfere with correctly configured clang-tidy setups
since an AllowedType with "::" would never match.

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

Reviewed-by: ymandel, hokein