Skip to content

Commit e4b424a

Browse files
authored
[CI] Disable Flang from pre-commit tests when Flang files are not touched on Windows Only (#93729)
Flang triggers some OOM on Windows CI right now. This is disruptive to MLIR and LLVM changes that don't touch Flang, as such we disable building Flang on Windows only for these PR that don't touch flang. The testing on Linux is unchanged, and the post-merge Windows testing is still fully covering here.
1 parent c3e6bd0 commit e4b424a

File tree

1 file changed

+14
-4
lines changed

1 file changed

+14
-4
lines changed

.ci/generate-buildkite-pipeline-premerge

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,10 @@ echo "Directories modified:" >&2
5353
echo "$modified_dirs" >&2
5454

5555
function compute-projects-to-test() {
56+
isForWindows=$1
57+
shift
5658
projects=${@}
59+
echo "isForWindows : $isForWindows ; projects: $projects " >&2
5760
for project in ${projects}; do
5861
echo "${project}"
5962
case ${project} in
@@ -63,9 +66,13 @@ function compute-projects-to-test() {
6366
done
6467
;;
6568
llvm)
66-
for p in bolt clang clang-tools-extra flang lld lldb mlir polly; do
69+
for p in bolt clang clang-tools-extra lld lldb mlir polly; do
6770
echo $p
6871
done
72+
# Flang is not stable in Windows CI at the moment
73+
if [[ $isForWindows == 0 ]]; then
74+
echo flang
75+
fi
6976
;;
7077
clang)
7178
for p in clang-tools-extra compiler-rt lldb cross-project-tests; do
@@ -76,7 +83,10 @@ function compute-projects-to-test() {
7683
echo libc
7784
;;
7885
mlir)
79-
echo flang
86+
# Flang is not stable in Windows CI at the moment
87+
if [[ $isForWindows == 0 ]]; then
88+
echo flang
89+
fi
8090
;;
8191
*)
8292
# Nothing to do
@@ -241,15 +251,15 @@ fi
241251
all_projects="bolt clang clang-tools-extra compiler-rt cross-project-tests flang libc libclc lld lldb llvm mlir openmp polly pstl"
242252
modified_projects="$(keep-modified-projects ${all_projects})"
243253

244-
linux_projects_to_test=$(exclude-linux $(compute-projects-to-test ${modified_projects}))
254+
linux_projects_to_test=$(exclude-linux $(compute-projects-to-test 0 ${modified_projects}))
245255
linux_check_targets=$(check-targets ${linux_projects_to_test} | sort | uniq)
246256
linux_projects=$(add-dependencies ${linux_projects_to_test} | sort | uniq)
247257

248258
linux_runtimes_to_test=$(compute-runtimes-to-test ${linux_projects_to_test})
249259
linux_runtime_check_targets=$(check-targets ${linux_runtimes_to_test} | sort | uniq)
250260
linux_runtimes=$(echo ${linux_runtimes_to_test} | sort | uniq)
251261

252-
windows_projects_to_test=$(exclude-windows $(compute-projects-to-test ${modified_projects}))
262+
windows_projects_to_test=$(exclude-windows $(compute-projects-to-test 1 ${modified_projects}))
253263
windows_check_targets=$(check-targets ${windows_projects_to_test} | sort | uniq)
254264
windows_projects=$(add-dependencies ${windows_projects_to_test} | sort | uniq)
255265

0 commit comments

Comments
 (0)