Skip to content

[SYCL][LIT] Fix regex in select_device LIT test #2690

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 1 commit into from
Oct 28, 2020
Merged

[SYCL][LIT] Fix regex in select_device LIT test #2690

merged 1 commit into from
Oct 28, 2020

Conversation

dm-vodopyanov
Copy link
Contributor

This patch adds two new special characters - [ and ] to
select_device LIT test. An escape symbol will be added to these
characters. This is because new OpenCL GPU runtime 20.42.18209 has
such symbols in its device name, and regex generated by this test
was w/o escape symbols for [ and ], which led to
CL_DEVICE_NOT_FOUND error in run time.

This patch adds two new special characters - ```[``` and ```]``` to
select_device LIT test. An escape symbol will be added to these
characters. This is because new OpenCL GPU runtime 20.42.18209 has
such symbols in its device name, and regex generated by this test
was w/o escape symbols for ```[``` and ```]```, which led to
```CL_DEVICE_NOT_FOUND``` error in run time.
@dm-vodopyanov dm-vodopyanov requested a review from a team as a code owner October 27, 2020 11:02
@dm-vodopyanov dm-vodopyanov requested a review from rbegam October 27, 2020 11:02
@rbegam
Copy link
Contributor

rbegam commented Oct 27, 2020

LGTM

@pvchupin pvchupin requested a review from romanovvlad October 27, 2020 22:17
@bader bader merged commit 28df843 into intel:sycl Oct 28, 2020
kbenzie added a commit to kbenzie/intel-llvm that referenced this pull request Feb 17, 2025
[L0 v2] add missing urCommadBufferEnqueueExp function
Chenyang-L pushed a commit that referenced this pull request Feb 18, 2025
[L0 v2] add missing urCommadBufferEnqueueExp function
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.

3 participants