[clang] [clang][scan-deps] Report a scanned TU's visible modules (PR #147969)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 10 08:56:03 PDT 2025


================
@@ -519,6 +528,8 @@ class FullDeps {
                       });
                     JOS.attributeArray("clang-module-deps",
                                        toJSONSorted(JOS, I.ClangModuleDeps));
+                    JOS.attributeArray("visible-clang-modules",
----------------
jansvoboda11 wrote:

I'm looking at it from the perspective that if I introduce a bug into the visibility logic, the test that I want to fail is `visible-modules.c`, not every other test under the sun. That reduces the noise and makes it easier to root-cause. The same applies when I'm working on some other feature (e.g. header search path pruning): if I introduce a bug, I'd like only for corresponding test to fail (`header-search-pruning-transitive.c`). If we feel the need for more coverage, I suggest adding more feature-specific tests.

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


More information about the cfe-commits mailing list