Skip to content

Commit

Permalink
Merge branch 'sycl' into e2e-split-ci-testing
Browse files Browse the repository at this point in the history
  • Loading branch information
ayylol committed Dec 20, 2024
2 parents 35f8e08 + 4fd899a commit c9eb5de
Show file tree
Hide file tree
Showing 10 changed files with 7 additions and 19 deletions.
2 changes: 1 addition & 1 deletion sycl/test-e2e/Basic/build_log.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// REQUIRES: opencl || level_zero, gpu, ocloc
// UNSUPPORTED: gpu-intel-dg1
// UNSUPPORTED: arch-intel_gpu_dg1
//
// RUN: %clangxx -fsycl -fsycl-targets=spir64_gen -Xsycl-target-backend=spir64_gen "-device dg1" %s -o %t.out
// RUN: env SYCL_RT_WARNING_LEVEL=2 %{run} %t.out 2>&1 | FileCheck %s
Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/Basic/query.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// RUN: %{build} -o %t.out
// RUN: %{run-unfiltered-devices} env ONEAPI_DEVICE_SELECTOR="*:gpu" %t.out

// REQUIRES: gpu-intel-dg1
// REQUIRES: arch-intel_gpu_dg1
#include "query.hpp"
2 changes: 1 addition & 1 deletion sycl/test-e2e/ESIMD/dp4a.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
//
//===----------------------------------------------------------------------===//
// TODO enable on Windows
// REQUIRES: linux && gpu-intel-dg1
// REQUIRES: linux && arch-intel_gpu_dg1
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// REQUIRES: gpu-intel-dg1 || gpu-intel-dg2 || arch-intel_gpu_pvc
// REQUIRES: arch-intel_gpu_dg1 || gpu-intel-dg2 || arch-intel_gpu_pvc
//
// Check that full compilation works:
// RUN: %clangxx -DIMPL_SUBGROUP -fsycl -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %S/../dp4a.cpp -o %t.out
Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/InvokeSimd/Regression/dp4a.cpp
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// REQUIRES: gpu-intel-dg1 || gpu-intel-dg2 || arch-intel_gpu_pvc
// REQUIRES: arch-intel_gpu_dg1 || gpu-intel-dg2 || arch-intel_gpu_pvc
//
// Check that full compilation works:
// RUN: %{build} -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr -o %t.out
Expand Down
3 changes: 0 additions & 3 deletions sycl/test-e2e/Matrix/get_coordinate_ops.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_matrix

// XFAIL: !igc-dev && run-mode
// XFAIL-TRACKER: GSD-6376
// REQUIRES-INTEL-DRIVER: lin: 30049

// RUN: %{build} -o %t.out
Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/OptionalKernelFeatures/fp64-conv-emu-1.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
// UNSUPPORTED: cuda, hip
// UNSUPPORTED-REASON: FP64 emulation is an Intel specific feature.

// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_dg2_g10,intel_gpu_dg2_g11,intel_gpu_dg2_g12,intel_gpu_pvc,intel_gpu_mtl_h,intel_gpu_mtl_u -fsycl-fp64-conv-emu -O0 %s -o %t.out
// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_dg2_g10,intel_gpu_dg2_g11,intel_gpu_dg2_g12,intel_gpu_pvc,intel_gpu_mtl_h,intel_gpu_mtl_u -fsycl-fp64-conv-emu %O0 %s -o %t.out
// RUN: %{run} %t.out

// Tests that aspect::fp64 is not emitted correctly when -fsycl-fp64-conv-emu
Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/OptionalKernelFeatures/fp64-conv-emu-2.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
// UNSUPPORTED: cuda, hip
// UNSUPPORTED-REASON: FP64 emulation is an Intel specific feature.

// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_dg2_g10,intel_gpu_dg2_g11,intel_gpu_dg2_g12,intel_gpu_pvc,intel_gpu_mtl_h,intel_gpu_mtl_u -fsycl-fp64-conv-emu -O0 %s -o %t.out
// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_dg2_g10,intel_gpu_dg2_g11,intel_gpu_dg2_g12,intel_gpu_pvc,intel_gpu_mtl_h,intel_gpu_mtl_u -fsycl-fp64-conv-emu %O0 %s -o %t.out
// RUN: %{run} %t.out

#include <sycl/detail/core.hpp>
Expand Down
7 changes: 0 additions & 7 deletions sycl/test-e2e/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,6 @@ place. No new tests should use these features:

* **gpu-intel-gen11** - Intel GPU Gen11 availability;
* **gpu-intel-gen12** - Intel GPU Gen12 availability;
* **gpu-intel-dg1** - Intel GPU DG1 availability;
* **gpu-intel-dg2** - Intel GPU DG2 availability;
* **gpu-intel-pvc** - Intel GPU PVC availability;
* **gpu-intel-pvc-vg** - Intel GPU PVC-VG availability;
Expand All @@ -268,12 +267,6 @@ configure specific single test execution in the command line:
* **gpu_aot_target_opts** - defines additional options which are passed to AOT
compilation command line for GPU device. It can be also set by CMake variable
GPU_AOT_TARGET_OPTS. If not specified "-device *" value is used.
* **gpu-intel-dg1** - tells LIT infra that Intel GPU DG1 is present in the
system. It is developer / CI infra responsibility to make sure that the
device is available in the system. Tests requiring DG1 to run must use proper
device selector to ensure that. Use SYCL_DEVICE_ALLOWLIST or
ONEAPI_DEVICE_SELECTOR to get proper configuration (see
[EnvironmentVariables.md](https://github.com/intel/llvm/blob/sycl/sycl/doc/EnvironmentVariables.md));
* **gpu-intel-dg2** - tells LIT infra that Intel GPU DG2 is present in the
system. It is developer / CI infra responsibility to make sure that the
device is available in the system.
Expand Down
2 changes: 0 additions & 2 deletions sycl/test-e2e/lit.cfg.py
Original file line number Diff line number Diff line change
Expand Up @@ -168,8 +168,6 @@
config.available_features.add("gpu-intel-gen12")

# Intel GPU DEVICE availability
if lit_config.params.get("gpu-intel-dg1", False):
config.available_features.add("gpu-intel-dg1")
if lit_config.params.get("gpu-intel-dg2", False):
config.available_features.add("gpu-intel-dg2")
if lit_config.params.get("gpu-intel-pvc-vg", False):
Expand Down

0 comments on commit c9eb5de

Please sign in to comment.