[PATCH] D36892: [clang-tidy] check_clang_tidy.py: support CHECK-NOTES prefix

[Alexander Kornienko via Phabricator via cfe-commits]

alexfh added a comment.

It looks like some existing tests could benefit from this:

  $ grep -R -l 'note: ' test/clang-tidy/
  test/clang-tidy/bugprone-forward-declaration-namespace.cpp
  test/clang-tidy/llvm-twine-local.cpp
  test/clang-tidy/overlapping.cpp
  test/clang-tidy/google-readability-nested-namespace-comments.cpp
  test/clang-tidy/bugprone-suspicious-enum-usage-strict.cpp
  test/clang-tidy/cert-static-object-exception.cpp
  test/clang-tidy/fix.cpp
  test/clang-tidy/google-readability-namespace-comments.cpp
  test/clang-tidy/cppcoreguidelines-avoid-goto.cpp
  test/clang-tidy/performance-move-constructor-init.cpp
  test/clang-tidy/fuchsia-default-arguments.cpp
  test/clang-tidy/readability-misleading-indentation.cpp
  test/clang-tidy/readability-inconsistent-declaration-parameter-name-macros.cpp
  test/clang-tidy/fix-errors.cpp
  test/clang-tidy/bugprone-use-after-move.cpp
  test/clang-tidy/cppcoreguidelines-owning-memory-containers.cpp
  test/clang-tidy/bugprone-argument-comment.cpp
  test/clang-tidy/cppcoreguidelines-owning-memory.cpp
  test/clang-tidy/readability-function-size.cpp
  test/clang-tidy/hicpp-exception-baseclass.cpp
  test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp
  test/clang-tidy/macros.cpp
  test/clang-tidy/readability-container-size-empty.cpp
  test/clang-tidy/Inputs/overlapping/o.h
  test/clang-tidy/bugprone-forwarding-reference-overload.cpp

If you're interested in changing some of these tests to use CHECK-NOTES, there's no need to wait for a resolution on https://reviews.llvm.org/D36836 (which, unfortunately, may take really long). However, please update the relevant section of docs/clang-tidy/index.rst.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D36892