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

[SYCL] Disable failing test #658

Merged
merged 1 commit into from
Dec 22, 2021
Merged

[SYCL] Disable failing test #658

merged 1 commit into from
Dec 22, 2021

Conversation

alexbatashev
Copy link

Blocks pre-commit testing

@alexbatashev alexbatashev requested a review from bader December 22, 2021 17:07
@alexbatashev alexbatashev requested a review from a team as a code owner December 22, 2021 17:07
@bader
Copy link

bader commented Dec 22, 2021

@alexbatashev, it looks like @vladimirlaz has a fix #659.
If #659 passes pre-commit, I think it's better to go with it.
OTOH, we need to fix intel/llvm pre-commit validation ASAP.
@vladimirlaz, are you okay if we merge this this PR right away and turn the test on back in #659?

@vladimirlaz
Copy link

@alexbatashev, it looks like @vladimirlaz has a fix #659. If #659 passes pre-commit, I think it's better to go with it. OTOH, we need to fix intel/llvm pre-commit validation ASAP. @vladimirlaz, are you okay if we merge this this PR right away and turn the test on back in #659?

Ok for me

@bader bader merged commit 50d1c28 into intel:intel Dec 22, 2021
@yubingex007-a11y yubingex007-a11y mentioned this pull request Dec 23, 2021
Merged
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
@alexbatashev
[SYCL] Disable failing test (intel/llvm-test-suite#658)
8778e38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bader bader Awaiting requested review from bader

@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.
3 participants
@alexbatashev @bader @vladimirlaz