Skip to content

Commit

Permalink
[SYCL][E2E] Remove XFAIL from tests; fix multi-device config (intel#1…
Browse files Browse the repository at this point in the history
…4571)

Resolves intel#14562.
  • Loading branch information
frasercrmck authored and smanna12 committed Jul 16, 2024
1 parent 5401bac commit ac20312
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 9 deletions.
6 changes: 2 additions & 4 deletions sycl/test-e2e/BFloat16/bfloat16_builtins.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,15 @@
// + below sm_80 always uses generic impls

// DEFINE: %{mathflags} = %if cl_options %{/clang:-fno-fast-math%} %else %{-fno-fast-math%}
// RUN: %clangxx -fsycl -fsycl-targets=%{sycl_triple} %if any-device-is-cuda %{ -Xsycl-target-backend --cuda-gpu-arch=sm_80 %} %s -o %t.out %{mathflags}
// RUN: %clangxx -fsycl -fsycl-targets=%{sycl_triple} %if any-device-is-cuda %{ -Xsycl-target-backend=nvptx64-nvidia-cuda --cuda-gpu-arch=sm_80 %} %s -o %t.out %{mathflags}
// RUN: %{run} %t.out

// Test "new" (ABI breaking) for all platforms ( sm_80/native if CUDA )
// RUN: %if preview-breaking-changes-supported %{ %clangxx -fsycl -fpreview-breaking-changes -fsycl-targets=%{sycl_triple} %if any-device-is-cuda %{ -Xsycl-target-backend --cuda-gpu-arch=sm_80 %} %s -o %t2.out %{mathflags} %}
// RUN: %if preview-breaking-changes-supported %{ %clangxx -fsycl -fpreview-breaking-changes -fsycl-targets=%{sycl_triple} %if any-device-is-cuda %{ -Xsycl-target-backend=nvptx64-nvidia-cuda --cuda-gpu-arch=sm_80 %} %s -o %t2.out %{mathflags} %}
// RUN: %if preview-breaking-changes-supported %{ %{run} %t2.out %}

// Currently the feature isn't supported on FPGA.
// UNSUPPORTED: accelerator
// FIXME: enable opaque pointers support on CPU.
// XFAIL: cpu
#include "bfloat16_builtins.hpp"

int main() {
Expand Down
4 changes: 1 addition & 3 deletions sycl/test-e2e/BFloat16/bfloat16_builtins_cuda_generic.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,11 @@
// DEFINE: %{mathflags} = %if cl_options %{/clang:-fno-fast-math%} %else %{-fno-fast-math%}

// If CUDA, test "new" again for sm_75/generic
// RUN: %if any-device-is-cuda %{ %if preview-breaking-changes-supported %{ %clangxx -fsycl -fpreview-breaking-changes -fsycl-targets=%{sycl_triple} -Xsycl-target-backend --cuda-gpu-arch=sm_75 %s -o %t3.out %{mathflags} %} %}
// RUN: %if any-device-is-cuda %{ %if preview-breaking-changes-supported %{ %clangxx -fsycl -fpreview-breaking-changes -fsycl-targets=%{sycl_triple} -Xsycl-target-backend=nvptx64-nvidia-cuda --cuda-gpu-arch=sm_75 %s -o %t3.out %{mathflags} %} %}
// RUN: %if any-device-is-cuda %{ %if preview-breaking-changes-supported %{ %{run} %t3.out %} %}

// Currently the feature isn't supported on FPGA.
// UNSUPPORTED: accelerator
// FIXME: enable opaque pointers support on CPU.
// XFAIL: cpu
#include "bfloat16_builtins.hpp"

int main() {
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/BFloat16/bfloat16_type.cpp
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// RUN: %if any-device-is-cuda %{ %{build} -DUSE_CUDA_SM80=1 -Xsycl-target-backend --cuda-gpu-arch=sm_80 -o %t.out %}
// RUN: %if any-device-is-cuda %{ %{build} -DUSE_CUDA_SM80=1 -Xsycl-target-backend=nvptx64-nvidia-cuda --cuda-gpu-arch=sm_80 -o %t.out %}
// RUN: %if cuda %{ %{run} %t.out %}
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
Expand All @@ -7,7 +7,6 @@
// UNSUPPORTED: accelerator

// FIXME: enable opaque pointers support on CPU.
// XFAIL: cpu

//==----------- bfloat16_type.cpp - SYCL bfloat16 type test ----------------==//
//
Expand Down

0 comments on commit ac20312

Please sign in to comment.