This revision was automatically updated to reflect the committed changes.
Closed by commit rL288892: Compilation database test: don't try to output to
CWD (authored by sammccall).
Changed prior to commit:
https://reviews.llvm.org/D27504?vs=80553&id=80554#toc
Repository:
rL LLVM
https://revi
sammccall marked an inline comment as done.
sammccall added a comment.
Thanks! That makes sense.
https://reviews.llvm.org/D27504
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sammccall updated this revision to Diff 80553.
sammccall added a comment.
Hoist cd %T to avoid repeating it.
https://reviews.llvm.org/D27504
Files:
test/Driver/compilation_database.c
Index: test/Driver/compilation_database.c
==
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
Annoying. This test really shouldn't generate binary object files.
Can we check in this patch now to unblock stuff and then look into how to do a
dry run with -MJ? Usually driver tests use `
sammccall added a comment.
bkramer: summoning my lit guru for guidance, is there a better way?
https://reviews.llvm.org/D27504
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sammccall created this revision.
sammccall added a reviewer: bkramer.
sammccall added subscribers: cfe-commits, joerg.
Write output from compilation database test to %T rather than the working dir.
Sometimes CWD isn't writable!
Also specify no-canonical-prefixes so that clang has 'clang' in the na
