Skip to content

[SYCL][CUDA] Add missing macro to interop-backend-traits.cpp #7603

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

JackAKirk
Copy link
Contributor

@JackAKirk JackAKirk commented Dec 1, 2022

This test generally uses traits of different backends when compiling using the default spirv triple. It seems this means that the test manually defines some macros that are usually defined by the backend. For the ext_oneapi_cuda case a macro was missing leading to the test failure. I have added this missing required macro.

Fixes this post-commit windows failure: https://github.com/intel/llvm/actions/runs/3585225542/jobs/6032932018

Signed-off-by: JackAKirk [email protected]

Signed-off-by: JackAKirk <[email protected]>
@pvchupin
Copy link
Contributor

pvchupin commented Dec 1, 2022

Unresolved Tests (5):
  SYCL :: Assert/assert_in_kernels.cpp
  SYCL :: Assert/assert_in_multiple_tus.cpp
  SYCL :: Assert/assert_in_multiple_tus_one_ndebug.cpp
  SYCL :: Assert/assert_in_one_kernel.cpp
  SYCL :: Assert/assert_in_simultaneously_multiple_tus_one_ndebug.cpp

I'll look into that.

@pvchupin pvchupin merged commit a578c81 into intel:sycl Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants