Author: Haojian Wu Date: 2023-09-01T08:23:39+02:00 New Revision: dc2d2f9ba1bc3e72d011c6203eb986e8d0f5ca32
URL: https://github.com/llvm/llvm-project/commit/dc2d2f9ba1bc3e72d011c6203eb986e8d0f5ca32 DIFF: https://github.com/llvm/llvm-project/commit/dc2d2f9ba1bc3e72d011c6203eb986e8d0f5ca32.diff LOG: [clangd] Respect IWYU keep pragma for standard headers. see the issue https://github.com/llvm/llvm-project/issues/64191 Differential Revision: https://reviews.llvm.org/D156650 (cherry picked from commit dcb28244faa88cb566a852533790bcac75daaa0f) Added: Modified: clang-tools-extra/clangd/IncludeCleaner.cpp clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp Removed: ################################################################################ diff --git a/clang-tools-extra/clangd/IncludeCleaner.cpp b/clang-tools-extra/clangd/IncludeCleaner.cpp index 9708c67ca2883c..b2c04ac4d54636 100644 --- a/clang-tools-extra/clangd/IncludeCleaner.cpp +++ b/clang-tools-extra/clangd/IncludeCleaner.cpp @@ -70,6 +70,8 @@ bool isIgnored(llvm::StringRef HeaderPath, HeaderFilter IgnoreHeaders) { bool mayConsiderUnused( const Inclusion &Inc, ParsedAST &AST, const include_cleaner::PragmaIncludes *PI) { + if (PI && PI->shouldKeep(Inc.HashLine + 1)) + return false; // FIXME(kirillbobyrev): We currently do not support the umbrella headers. // System headers are likely to be standard library headers. // Until we have good support for umbrella headers, don't warn about them. @@ -81,8 +83,6 @@ bool mayConsiderUnused( AST.getIncludeStructure().getRealPath(HID)); assert(FE); if (PI) { - if (PI->shouldKeep(Inc.HashLine + 1)) - return false; // Check if main file is the public interface for a private header. If so we // shouldn't diagnose it as unused. if (auto PHeader = PI->getPublic(*FE); !PHeader.empty()) { diff --git a/clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp b/clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp index c55351fb1f91d8..83a7c45df1695f 100644 --- a/clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp +++ b/clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp @@ -76,6 +76,8 @@ TEST(IncludeCleaner, StdlibUnused) { auto TU = TestTU::withCode(R"cpp( #include <list> #include <queue> + #include <vector> // IWYU pragma: keep + #include <string> // IWYU pragma: export std::list<int> x; )cpp"); // Layout of std library impl is not relevant. @@ -84,10 +86,13 @@ TEST(IncludeCleaner, StdlibUnused) { namespace std { template <typename> class list {}; template <typename> class queue {}; + template <typename> class vector {}; } )cpp"; TU.AdditionalFiles["list"] = "#include <bits>"; TU.AdditionalFiles["queue"] = "#include <bits>"; + TU.AdditionalFiles["vector"] = "#include <bits>"; + TU.AdditionalFiles["string"] = "#include <bits>"; TU.ExtraArgs = {"-isystem", testRoot()}; auto AST = TU.build(); IncludeCleanerFindings Findings = computeIncludeCleanerFindings(AST); _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
