Skip to content

Commit 7d3ac99

Browse files
authored
[SYCL][InvokeSimd] Fix two XFAIL tests (#14906)
For reference_argument.cpp, the test `RUN` line was wrong, and I added the real error message to the `CHECK` string. For `simd32_platform_error.cpp`, all Intel GPUs we test on report as support SIMD32 (for the ones that don't in hardware, it is probably emulated but it definitely works), so they don't throw this error so we can't test it, so just remove the test. Closes: #14643 Closes: #14644 Signed-off-by: Sarnie, Nick <[email protected]>
1 parent 1edc943 commit 7d3ac99

File tree

2 files changed

+2
-35
lines changed

2 files changed

+2
-35
lines changed

sycl/test-e2e/InvokeSimd/Spec/clang_run_error/reference_argument.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
1-
// RUN: not %{build} -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %s -o %t.out 2>&1 | FileCheck
1+
// RUN: not %{build} -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %s -o %t.out 2>&1 | FileCheck %s
22
//
3-
// TODO FIXME: Currently compile fail with "no or multiple invoke_simd targets
4-
// found"
5-
// XFAIL: gpu
63
/*
74
* Test case specification: Test and report errors if reference argument is
85
* passed to invoked ESIMD function
@@ -79,7 +76,7 @@ int main(void) {
7976

8077
float res = invoke_simd(sg, SIMD_CALLEE<int_ref>, uniform{A},
8178
B[wi_id], uniform{i});
82-
// CHECK: TODO FIXME
79+
// CHECK: invoke_simd does not support callables with reference arguments
8380
C[wi_id] = res;
8481
});
8582
});

sycl/test-e2e/InvokeSimd/Spec/simd_size/simd32_platform_error.cpp

Lines changed: 0 additions & 30 deletions
This file was deleted.

0 commit comments

Comments
 (0)