[PATCH] D133617: [Clang][ScanDeps] Change multiple-commands.c test to use -fmodules-cache-path on implicit builds
Ben Langmuir via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 9 15:28:06 PDT 2022
benlangmuir accepted this revision.
benlangmuir added a comment.
This revision is now accepted and ready to land.
Thanks for catching this!
================
Comment at: clang/test/ClangScanDeps/multiple-commands.c:11
// RUN: split-file %s %t
-// RUN: sed -e "s|DIR|%/t|g" %t/cdb.json.in > %t/cdb.json
+// RUN: sed -e "s|DIR|%/t|g" -e "s|CACHE_PATH|%t.modcache|g" %t/cdb.json.in > %t/cdb.json
----------------
Nit: we could just put it inside DIR/modcache and not need a second substitution.
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