[libcxx-commits] [PATCH] D127587: [libc++] Use explicit module cache path in tests

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Aug 31 08:38:56 PDT 2023


ldionne commandeered this revision.
ldionne edited reviewers, added: abrachet; removed: ldionne.
ldionne added a comment.

[Github PR transition cleanup]

I think this is reasonable enough that we don't need to look into changing anything. I think it would be better to define a substitution like `%{modules-cache}` which is global to the test suite and then use `-fmodules-cache-path=%{modules-cache}` here in this test and also in `params.py` (where we currently define a modules cache path for clang-modules builds).

@abrachet If you want to investigate this, please open that review as a GH PR and we'll have a look. But otherwise the current state is acceptable as well.

I will commandeer this to close it to clear the review queue.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127587/new/

https://reviews.llvm.org/D127587



More information about the libcxx-commits mailing list