================
@@ -1,23 +1,10 @@
 // RUN: rm -rf %t
 // RUN: split-file %s %t
 
-//--- missing_tu.json.in
-[{
-  "directory": "DIR",
-  "command": "clang -fsyntax-only DIR/missing_tu.c",
-  "file": "DIR/missing_tu.c"
-}]
-//--- missing_header.json.in
-[{
-  "directory": "DIR",
-  "command": "clang -fsyntax-only DIR/missing_header.c",
-  "file": "DIR/missing_header.c"
-}]
 //--- missing_header.c
 #include "missing.h"
 
-// RUN: sed -e "s|DIR|%/t|g" %t/missing_tu.json.in > %t/missing_tu.json
-// RUN: not clang-scan-deps -compilation-database %t/missing_tu.json 
2>%t/missing_tu.errs
+// RUN: not clang-scan-deps -- "clang" %t/missing_tu.c 2>%t/missing_tu.errs
----------------
jansvoboda11 wrote:

I just tried double-checking what `%clang` expands to and I was wrong, it just 
expands to the Clang binary in the build directory, so I think I can remove the 
quote workaround.

However, both `"clang"` and `%clang` expand to `-cc1` invocations that refer to 
system directories: `-isystem /usr/local/include -internal-externc-isystem 
/usr/include`. It would be nice to have something that cleans these out (like 
`%clang_cc1` does), but I consider that out of scope for this patch. (Not 
functional change from in this aspect anyways.)

https://github.com/llvm/llvm-project/pull/88764
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to