Skip to content

Commit a86340c

Browse files
author
Colin Davidson
committed
[SYCL][E2E][NATIVE_CPU] Add Native CPU e2e PR testing
The Native CPU work is being passed onto a team in Intel now that all of Native CPU is independent of the oneAPI Construction Kit. Native CPU does not pass all of E2E testing, either through bugs or unimplemented functionality. All are currently being marked as UNSUPPORTED until these failures are processed and tracked by the new team. This also adds a separate job for E2E build and run for Native CPU, as it cannot be added to the prebuilt due to build fails.
1 parent 3274ca1 commit a86340c

File tree

54 files changed

+175
-39
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+175
-39
lines changed

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

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,42 @@ jobs:
6262
e2e_binaries_artifact: e2e_bin
6363
e2e_binaries_preview_artifact: e2e_bin_preview
6464

65+
# Build and run native cpu e2e tests separately as cannot currently
66+
# build all the e2e tests
67+
build_run_native_cpu_e2e_tests:
68+
if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }}
69+
runs-on: [Linux, build]
70+
needs: [build]
71+
container:
72+
image: ghcr.io/intel/llvm/sycl_ubuntu2404_nightly:latest
73+
options: -u 1001:1001
74+
steps:
75+
- uses: actions/checkout@v4
76+
with:
77+
sparse-checkout: |
78+
devops/
79+
80+
# download build artefact
81+
- name: Download toolchain
82+
uses: actions/download-artifact@v4
83+
with:
84+
name: sycl_linux_default
85+
- name: Extract SYCL toolchain
86+
shell: bash
87+
run: |
88+
mkdir toolchain
89+
tar -xf llvm_sycl.tar.zst -C toolchain
90+
rm llvm_sycl.tar.zst
91+
- name: Build and run E2E tests
92+
uses: ./devops/actions/run-tests/e2e
93+
with:
94+
ref: ${{ inputs.ref || github.sha }}
95+
testing_mode: build-only
96+
target_devices: native_cpu
97+
sycl_compiler: $GITHUB_WORKSPACE/toolchain/bin/clang++
98+
extra_lit_opts: --param sycl_build_targets="native_cpu"
99+
extra_cmake_args: -DSYCL_TEST_E2E_TARGETS="native_cpu:cpu" -DSYCL_TEST_E2E_STANDALONE=ON
100+
65101
# If a PR changes CUDA adapter, run the build on Ubuntu 22.04 as well.
66102
# Ubuntu 22.04 container has CUDA 12.1 installed while Ubuntu 24.0 image
67103
# has CUDA 12.6.1 installed.

‎sycl/test-e2e/Adapters/enqueue-arg-order-image.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// REQUIRES: aspect-ext_intel_legacy_image
2-
// UNSUPPORTED: target-amd
2+
// UNSUPPORTED: target-amd || target-native_cpu
33
// UNSUPPORTED-INTENDED: Legacy images aren't supported on AMD, but also fail
44
// to compile. Bindless images should be used instead.
55

‎sycl/test-e2e/Adapters/level_zero/lit.local.cfg‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# Currently mark Native CPU as unsupported, does not compile and should be investigated and tracked post team transfer
2+
config.unsupported_features += ['target-native_cpu']

‎sycl/test-e2e/Adapters/queue_submit_mode.cpp‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
// REQUIRES: gpu, level_zero
22
// UNSUPPORTED: level_zero_v2_adapter
33
// UNSUPPORTED-INTENDED: v2 adapter does support regular cmd lists
4+
// UNSUPPORTED: target-native_cpu
5+
// UNSUPPORTED-INTENDED: Currently mark Native CPU as unsupported, should be
6+
// investigated and tracked post team transfer
47
// RUN: %{build} %level_zero_options -o %t.out
58
// RUN: env %{l0_leak_check} SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=0 %{run} %t.out 0 2>&1 | FileCheck %s --check-prefixes=CHECK-STD
69
// RUN: env %{l0_leak_check} SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 %{run} %t.out 1 2>&1 | FileCheck %s --check-prefixes=CHECK-IMM

‎sycl/test-e2e/AddressSanitizer/lit.local.cfg‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ config.unsupported_features += ['spirv-backend']
2121
# https://github.com/intel/llvm/issues/16920
2222
config.unsupported_features += ['arch-intel_gpu_bmg_g21']
2323

24+
# AddressSanitizer currently not supported for Native CPU
25+
config.unsupported_features += ['target-native_cpu']
2426
unsupported_san_flags = [
2527
# "-fsanitize=address",
2628
"-fsanitize=memory",

‎sycl/test-e2e/Assert/check_resource_leak.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
// RUN: %{build} -Wno-error=#warnings -o %t.out
22
// RUN: %{run} %t.out
33

4-
// Device globals aren't supported on opencl:gpu yet.
5-
// UNSUPPORTED: opencl && gpu
4+
// Device globals aren't supported on opencl:gpu or Native CPU yet.
5+
// UNSUPPORTED: (opencl && gpu) || target-native_cpu
66

77
// TODO: Fails at JIT compilation for some reason.
88
// UNSUPPORTED: hip

‎sycl/test-e2e/BFloat16/lit.local.cfg‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# BFloat16 is not supported on Native CPU
2+
config.unsupported_features += ['target-native_cpu']

‎sycl/test-e2e/Basic/built-ins/marray_math.cpp‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
// UNSUPPORTED: target-native_cpu
2+
// UNSUPPORTED-INTENDED: Currently mark Native CPU as unsupported, should be
3+
// investigated and tracked post team transfer
4+
15
// DEFINE: %{mathflags} = %if cl_options %{/clang:-fno-fast-math%} %else %{-fno-fast-math%}
26

37
// RUN: %{build} %{mathflags} -o %t.out

‎sycl/test-e2e/Basic/image/lit.local.cfg‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
# https://github.com/KhronosGroup/SPIRV-Headers/issues/487
44
# After the issue is resolved we will re-enable Image support.
55
#
6-
# Legacy images aren't supported on AMD and also don't compile, so mark them
6+
# Legacy images aren't supported on AMD and Native CPU and also don't compile, so mark them
77
# unsupported here. Bindless images should be used instead.
8-
config.unsupported_features += ['spirv-backend', 'target-amd']
8+
config.unsupported_features += ['spirv-backend', 'target-amd', 'target-native_cpu']

‎sycl/test-e2e/Basic/vector/load_store.cpp‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
// UNSUPPORTED: target-native_cpu
2+
// UNSUPPORTED-INTENDED: Currently mark Native CPU as unsupported, should be
3+
// investigated and tracked post team transfer
4+
15
// RUN: %{build} -o %t.out
26
// RUN: %{run} %t.out
37

0 commit comments

Comments
 (0)