Skip to content

Commit d93ba81

Browse files
ldionnetru
authored andcommitted
[libc++] Deflake the Clang Modules CI job
This re-introduces the workaround that had been introduced in d7ca140 and then removed in 0c0628c, since it seems like it is needed after all. Differential Revision: https://reviews.llvm.org/D157319 (cherry picked from commit d2a61db)
1 parent 63e39e8 commit d93ba81

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

libcxx/utils/libcxx/test/params.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,10 @@ def getModuleFlag(cfg, enable_modules):
121121
AddFeature("modules-build"),
122122
AddCompileFlag("-fmodules"),
123123
AddCompileFlag("-fcxx-modules"), # AppleClang disregards -fmodules entirely when compiling C++. This enables modules for C++.
124+
# Note: We use a custom modules cache path to make sure that we don't reuse
125+
# the default one, which can be shared across CI builds with different
126+
# configurations.
127+
AddCompileFlag(lambda cfg: f"-fmodules-cache-path={cfg.test_exec_root}/ModuleCache"),
124128
]
125129
if enable_modules == "clang"
126130
else [

0 commit comments

Comments
 (0)