[PATCH] D133617: [Clang][ScanDeps] Change multiple-commands.c test to use -fmodules-cache-path on implicit builds
Jan Svoboda via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 9 15:29:54 PDT 2022
jansvoboda11 accepted this revision.
jansvoboda11 added a comment.
LGTM, thanks! Though we usually put the cache directory into `%t/cache` (aka `DIR/cache` in the JSON file) in other scanner tests, which would allow you to drop the second `sed` expression. (And you'd want to `rm -rf %t` instead.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133617/new/
https://reviews.llvm.org/D133617
More information about the cfe-commits
mailing list