[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