[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 09:00:20 PDT 2025
================
@@ -519,6 +528,8 @@ class FullDeps {
});
JOS.attributeArray("clang-module-deps",
toJSONSorted(JOS, I.ClangModuleDeps));
+ JOS.attributeArray("visible-clang-modules",
----------------
jansvoboda11 wrote:
FWIW we've historically not been great at this - for example lots of tests check the set of input files even though they only really want to check the command-line. I've typically done this so that `FileCheck` can run line-by-line via `CHECK-NEXT` in order to avoid some sharp edges with using `CHECK` in a structured input. This has bitten me later, for example when making sure all input paths we report are absolute and normalized, which has caused lots of unnecessary churn. Looking back, I think we should've introduced a flag to `clang-scan-deps` to skip outputting the input files and simplify most of our tests that don't care about input files.
https://github.com/llvm/llvm-project/pull/147969
More information about the cfe-commits
mailing list