[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