[all-commits] [llvm/llvm-project] 665bfb: Reland "[clang-tidy] modernize-deprecated-headers ...

Balazs Benics via All-commits all-commits at lists.llvm.org
Fri May 20 12:13:05 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 665bfbb98daa0e40a2e6cecfbbfd6949980d8f2f
      https://github.com/llvm/llvm-project/commit/665bfbb98daa0e40a2e6cecfbbfd6949980d8f2f
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-05-20 (Fri, 20 May 2022)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.cpp
    M clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.h
    M clang-tools-extra/docs/ReleaseNotes.rst
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/modernize-deprecated-headers/mylib.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/modernize-deprecated-headers/mysystemlib.h
    A clang-tools-extra/test/clang-tidy/checkers/modernize-deprecated-headers-extern-c.cpp

  Log Message:
  -----------
  Reland "[clang-tidy] modernize-deprecated-headers check should respect extern "C" blocks""

This partially reverts commit e8cae487022c2216182ae1ec24f248f287a614b7.
Changes since that commit:
 - Use `SourceManager::isBeforeInTranslationUnit` instead of the fancy
   decomposed decl logarithmic search.
 - Add a test for including a system header containing a deprecated
   include.
 - Add `REQUIRES: system-linux` clause to the test.

Reviewed By: LegalizeAdulthood, whisperity

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




More information about the All-commits mailing list