[all-commits] [llvm/llvm-project] d24920: Revert "Re-Revert "clang-tidy: introduce readabili...

[Saleem Abdulrasool via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d249200fa7d540fb0b3ddc065575293e1da11107
      https://github.com/llvm/llvm-project/commit/d249200fa7d540fb0b3ddc065575293e1da11107
  Author: Saleem Abdulrasool <compnerd at compnerd.org>
  Date:   2021-09-15 (Wed, 15 Sep 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/readability/CMakeLists.txt
    A clang-tools-extra/clang-tidy/readability/ContainerDataPointerCheck.cpp
    A clang-tools-extra/clang-tidy/readability/ContainerDataPointerCheck.h
    M clang-tools-extra/clang-tidy/readability/ReadabilityTidyModule.cpp
    M clang-tools-extra/docs/ReleaseNotes.rst
    A clang-tools-extra/docs/clang-tidy/checks/readability-data-pointer.rst
    A clang-tools-extra/test/clang-tidy/checkers/readability-container-data-pointer.cpp

  Log Message:
  -----------
  Revert "Re-Revert "clang-tidy: introduce readability-containter-data-pointer check""

This reverts commit 626586fc253c6f032aedb325dba6b1ff3f11875e.

Tweak the test for Windows.  Windows defaults to delayed template
parsing, which resulted in the main template definition not registering
the test on Windows.  Process the file with the additional
`-fno-delayed-template-parsing` flag to change the default beahviour.
Additionally, add an extra check for the fix it and use a more robust
test to ensure that the value is always evaluated.

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