Skip to content

Commit b97b714

Browse files
authored
[CI][E2E] Run E2E tests on NVIDIA with prebuilt binaries in precommit (#16813)
Reduces time for precommit nvidia job from around 20m to around 5m
1 parent 75a5e28 commit b97b714

File tree

6 files changed

+10
-7
lines changed

6 files changed

+10
-7
lines changed

.github/workflows/sycl-linux-build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -279,6 +279,7 @@ jobs:
279279
target_devices: all
280280
binaries_artifact: ${{ inputs.e2e_binaries_artifact }}
281281
cxx_compiler: $GITHUB_WORKSPACE/toolchain/bin/clang++
282+
extra_lit_opts: --param sycl_build_targets="spir;nvidia"
282283

283284
- name: Remove E2E tests before spirv-backend run
284285
if: ${{ inputs.e2e_binaries_artifact && always() && !cancelled() && steps.build.conclusion == 'success' }}

.github/workflows/sycl-linux-precommit.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,10 @@ jobs:
8686
target_devices: level_zero:gpu;opencl:gpu;opencl:cpu
8787
reset_intel_gpu: true
8888
extra_lit_opts: --param gpu-intel-gen12=True
89+
- name: NVIDIA/CUDA
90+
runner: '["Linux", "cuda"]'
91+
image_options: -u 1001 --gpus all --cap-add SYS_ADMIN
92+
target_devices: cuda:gpu
8993
- name: Intel Arc A-Series Graphics
9094
runner: '["Linux", "arc"]'
9195
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
@@ -143,10 +147,6 @@ jobs:
143147
fail-fast: false
144148
matrix:
145149
include:
146-
- name: NVIDIA/CUDA
147-
runner: '["Linux", "cuda"]'
148-
image_options: -u 1001 --gpus all --cap-add SYS_ADMIN
149-
target_devices: cuda:gpu
150150
- name: AMD/HIP
151151
runner: '["Linux", "amdgpu"]'
152152
image_options: -u 1001 --device=/dev/dri --device=/dev/kfd

sycl/test-e2e/Matrix/joint_matrix_tensorcores_sm72.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88

9-
// REQUIRES: cuda
9+
// REQUIRES: target-nvidia
1010
// RUN: %{build} -Xsycl-target-backend --cuda-gpu-arch=sm_72 -o %t.out
1111
// RUN: %{run} %t.out
1212
//

sycl/test-e2e/Matrix/joint_matrix_tensorcores_sm80.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88

9-
// REQUIRES: cuda
9+
// REQUIRES: target-nvidia
1010
// RUN: %{build} -Xsycl-target-backend --cuda-gpu-arch=sm_80 -o %t.out
1111
// RUN: %{run} %t.out
1212
//

sycl/test-e2e/Regression/kernel_bundle_ignore_sycl_external.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// RUN: %{build} -o %t.out
22
// RUN: %{run} %t.out
33
//
4-
// XFAIL: cuda
4+
// XFAIL: target-nvidia
55
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16417
66
// UNSUPPORTED: hip
77

sycl/test-e2e/WorkGroupScratchMemory/copy_dynamic_size.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
// RUN: %{build} -o %t.out
22
// RUN: %{run} %t.out
33
//
4+
// XFAIL: cuda
5+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16968
46

57
// UNSUPPORTED: gpu-intel-gen12
68
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/16072

0 commit comments

Comments
 (0)