[all-commits] [llvm/llvm-project] f47564: [clangd] IncludeCleaner: Skip non self-contained h...

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Fri Oct 29 08:57:56 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f47564ea87a502f5fc6320ba5b57e91186751b12
      https://github.com/llvm/llvm-project/commit/f47564ea87a502f5fc6320ba5b57e91186751b12
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2021-10-29 (Fri, 29 Oct 2021)

  Changed paths:
    M clang-tools-extra/clangd/IncludeCleaner.cpp
    M clang-tools-extra/clangd/IncludeCleaner.h
    M clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
    M clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp

  Log Message:
  -----------
  [clangd] IncludeCleaner: Skip non self-contained headers

Headers without include guards might have side effects or can be the files we
don't want to consider (e.g. tablegen ".inc" files). Skip them when translating
headers to the HeaderIDs that we will consider as unused.

Reviewed By: sammccall

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




More information about the All-commits mailing list