Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][CUDA] Add triple to build step and remove XFAIL: cuda #339

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

steffenlarsen
Copy link

SYCL/Regression/atomic_load.cpp and SYCL/Scheduler/MemObjRemapping.cpp should pass for the PI CUDA backend, but are missing the corresponding triple in the build step. These changes add the -fsycl-targets with the triple to these tests and reenable them for CUDA.

SYCL/Regression/atomic_load.cpp and SYCL/Scheduler/MemObjRemapping.cpp
should pass for the PI CUDA backend, but are missing the corresponding
triple in the build step. These changes add the -fsycl-targets with the
triple to these tests and renable them for CUDA.

Signed-off-by: Steffen Larsen <[email protected]>
@steffenlarsen steffenlarsen requested review from vladimirlaz and a team as code owners July 2, 2021 13:24
@steffenlarsen steffenlarsen mentioned this pull request Jul 2, 2021
@vladimirlaz vladimirlaz merged commit 2a3f12a into intel:intel Jul 6, 2021
smaslov-intel pushed a commit to smaslov-intel/llvm-test-suite that referenced this pull request Aug 12, 2021
SYCL/Regression/atomic_load.cpp and SYCL/Scheduler/MemObjRemapping.cpp
should pass for the PI CUDA backend, but are missing the corresponding
triple in the build step. These changes add the -fsycl-targets with the
triple to these tests and renable them for CUDA.

Signed-off-by: Steffen Larsen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…lvm-test-suite#339)

SYCL/Regression/atomic_load.cpp and SYCL/Scheduler/MemObjRemapping.cpp
should pass for the PI CUDA backend, but are missing the corresponding
triple in the build step. These changes add the -fsycl-targets with the
triple to these tests and renable them for CUDA.

Signed-off-by: Steffen Larsen <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants