[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