Skip to content

[SYCL] Fix device selector bug introduced in #12288 #12328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jan 9, 2024

Conversation

aelovikov-intel
Copy link
Contributor

@aelovikov-intel aelovikov-intel commented Jan 9, 2024

The very last commit in #12288
(f9a1377) accidentally removed one condition needed to add the device into allowed devices list. Restore it here.

The very last commit in that PR
(intel@f9a1377)
accidentally removed one condition needed to add the device into allowed
devices list. Restore it here.
Copy link
Contributor

github-actions bot commented Jan 9, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

Copy link
Contributor

@maarquitos14 maarquitos14 left a comment

Choose a reason for hiding this comment

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

Is there any test that caught the error? If not, should we add one?

Copy link
Contributor

@maarquitos14 maarquitos14 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for adding the test.

@aelovikov-intel
Copy link
Contributor Author

CUDA failure looks like some race condition in the CI itself, not a real failure for this PR. Previuos version passed at https://github.com/intel/llvm/actions/runs/7466771596.

@aelovikov-intel aelovikov-intel merged commit 7331d28 into intel:sycl Jan 9, 2024
@aelovikov-intel aelovikov-intel deleted the selector-fix branch January 9, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants