martong added a comment. The tests are getting better, thanks Endre!
================ Comment at: clang/test/Analysis/ctu-on-demand-parsing.c:5 +// RUN: cp "%S/Inputs/ctu-other.c" "%t/ctu-other.c" +// RUN: echo '[{"directory":"%t","command":"gcc -c -std=c89 -Wno-visibility ctu-other.c","file":"ctu-other.c"}]' | sed -e 's/\\/\\\\/g' > %t/compile_commands.json +// RUN: cd "%t" && %clang_extdef_map ctu-other.c > externalDefMap.txt ---------------- Why do we need the `-Wno-visibility` flag? ================ Comment at: clang/test/Analysis/ctu-on-demand-parsing.c:6 +// RUN: echo '[{"directory":"%t","command":"gcc -c -std=c89 -Wno-visibility ctu-other.c","file":"ctu-other.c"}]' | sed -e 's/\\/\\\\/g' > %t/compile_commands.json +// RUN: cd "%t" && %clang_extdef_map ctu-other.c > externalDefMap.txt +// RUN: cd "%t" && %clang_cc1 -triple x86_64-pc-linux-gnu -fsyntax-only -std=c89 -analyze \ ---------------- If we execute `clang_extdef_map` then we don't need `Inputs/ctu-other.c[pp].externalDefMap.on-the-fly.txt` files. Do we? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75665/new/ https://reviews.llvm.org/D75665 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits