Skip to content

Commit 84f961d

Browse files
authored
Merge pull request #1067 from bader/ccache-cleanup
[CI][NFC] Update ccache workaround comments
2 parents 91601b1 + 7bd2faf commit 84f961d

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.github/workflows/cts_ci.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,10 @@ jobs:
141141
key: ${{ matrix.sycl-impl }}-ccache-${{ github.sha }}
142142
restore-keys: |
143143
${{ matrix.sycl-impl }}-ccache-
144-
# Use ccache's "depend mode" to work around DPC++ issue (see https://github.com/intel/llvm/issues/5260)
145-
# This requires compilation with -MD, which is enabled because we use the Ninja generator
146-
# Using this mode should not have any practical disadvantages
144+
# Use ccache's "depend mode" to meet the GitHub Actions timeout of 6 hours with DPC++ compiler.
145+
# This is a workaround for the fact that ccache's default mode (i.e. "preprocessor mode") introduces significant overhead in case of the cache miss.
146+
# This requires compilation with -MD, which is enabled because we use the Ninja generator.
147+
# Using this mode should not have any practical disadvantages.
147148
- name: Set ccache environment variables
148149
run: |
149150
echo "CCACHE_DEPEND=1" >> "$GITHUB_ENV"

0 commit comments

Comments
 (0)