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

[SYCL] Add target to is_compatible test #1428

Merged
merged 1 commit into from
Dec 1, 2022
Merged

[SYCL] Add target to is_compatible test #1428

merged 1 commit into from
Dec 1, 2022

Conversation

steffenlarsen
Copy link

This commit should fix the is_compatible test for non-SPIRV targets.

@steffenlarsen
[SYCL] Add target to is_compatible test
199a06b 
This commit should fix the is_compatible test for non-SPIRV targets.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen requested a review from a team as a code owner December 1, 2022 09:40
AlexeySachkov
AlexeySachkov approved these changes Dec 1, 2022
View reviewed changes
Copy link

@AlexeySachkov AlexeySachkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should improve our CI ASAP. So many issues are being introduced which could have been caught by pre-commit in this repo

@steffenlarsen steffenlarsen merged commit 1dd04ee into intel:intel Dec 1, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
@steffenlarsen @bb-sycl
[SYCL] Add target to is_compatible test (intel#1428)
c99246a 
This commit should fix the is_compatible test for non-SPIRV targets.

Signed-off-by: Larsen, Steffen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
@steffenlarsen
[SYCL] Add target to is_compatible test (intel/llvm-test-suite#1428)
18fd4fe 
This commit should fix the is_compatible test for non-SPIRV targets.

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

@dm-vodopyanov dm-vodopyanov dm-vodopyanov approved these changes

@AlexeySachkov AlexeySachkov AlexeySachkov approved these changes

@KornevNikita KornevNikita KornevNikita approved these changes

@cperkinsintel cperkinsintel Awaiting requested review from cperkinsintel cperkinsintel is a code owner automatically assigned from intel/llvm-reviewers-runtime

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@steffenlarsen @dm-vodopyanov @AlexeySachkov @KornevNikita