[clang-tools-extra] [clangd] Add config option to allow detection of unused system headers (PR #87208)

kadir çetinkaya via cfe-commits cfe-commits at lists.llvm.org
Fri May 10 01:26:24 PDT 2024


================
@@ -135,6 +135,21 @@ TEST(IncludeCleaner, GetUnusedHeaders) {
                            Pointee(writtenInclusion("\"dir/unused.h\""))));
 }
 
+TEST(IncludeCleaner, SystemUnusedHeaders) {
+  auto TU = TestTU::withCode(R"cpp(
+    #include <system_header.h>
+    #include <system_unused.h>
+    SystemClass x;
+  )cpp");
+  TU.AdditionalFiles["system/system_header.h"] = guard("class SystemClass {};");
+  TU.AdditionalFiles["system/system_unused.h"] = guard("");
+  TU.ExtraArgs = {"-isystem", testPath("system")};
+  auto AST = TU.build();
+  IncludeCleanerFindings Findings = computeIncludeCleanerFindings(AST, true);
----------------
kadircet wrote:

can you also add a negative test? (i.e. when `AnalyzeSystemHeaders = false`, `UnusedIncludes` is empty)

https://github.com/llvm/llvm-project/pull/87208


More information about the cfe-commits mailing list