Skip to content

Commit d2a61db

Browse files
committed
[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
1 parent 15ea230 commit d2a61db

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
@@ -127,6 +127,10 @@ def getModuleFlag(cfg, enable_modules):
127127
AddFeature("modules-build"),
128128
AddCompileFlag("-fmodules"),
129129
AddCompileFlag("-fcxx-modules"), # AppleClang disregards -fmodules entirely when compiling C++. This enables modules for C++.
130+
# Note: We use a custom modules cache path to make sure that we don't reuse
131+
# the default one, which can be shared across CI builds with different
132+
# configurations.
133+
AddCompileFlag(lambda cfg: f"-fmodules-cache-path={cfg.test_exec_root}/ModuleCache"),
130134
]
131135
if enable_modules == "clang"
132136
else [

0 commit comments

Comments
 (0)