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 <steffen.larsen@intel.com>
@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 <steffen.larsen@intel.com>
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 <steffen.larsen@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

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

3 more reviewers

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

@AlexeySachkov AlexeySachkov AlexeySachkov approved these changes

@KornevNikita KornevNikita KornevNikita approved these changes

Reviewers whose approvals may not affect merge requirements

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