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

[SYCL] [CUDA][HIP] Disable sycl-host-compiler test due to flaky failures #1252

Merged

Conversation

romanovvlad
Copy link

No description provided.

@romanovvlad romanovvlad requested a review from a team as a code owner September 12, 2022 13:57
steffenlarsen
steffenlarsen previously approved these changes Sep 12, 2022
@npmiller
Copy link

You should be able to fix these by simply adding -fsycl-targets=%sycl_triple to the RUN: line.

CUDA and HIP don't support SPIR-V, only ahead of time compilation so just -fsycl without -fsycl-targets doesn't work.

@romanovvlad romanovvlad merged commit 4b0ccd1 into intel:intel Sep 12, 2022
steffenlarsen added a commit to steffenlarsen/llvm-test-suite that referenced this pull request Sep 21, 2022
Similar to the changes in
intel#1252, the windows version
of the host compiler test needs the triple specified to work on certain
backends. This commit adds the triple flag to the compilation line of
the test.

Signed-off-by: Larsen, Steffen <[email protected]>
steffenlarsen added a commit that referenced this pull request Sep 21, 2022
…1282)

Similar to the changes in
#1252, the windows version
of the host compiler test needs the triple specified to work on certain
backends. This commit adds the triple flag to the compilation line of
the test.

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

Similar to the changes in
intel/llvm-test-suite#1252, the windows version
of the host compiler test needs the triple specified to work on certain
backends. This commit adds the triple flag to the compilation line of
the test.

Signed-off-by: Larsen, Steffen <[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.

3 participants