Skip to content

Revert "[CI][Deps] Uplift CPU/FPGAEMU RT version to 2023.15.3.0.20" #8952

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 5, 2023
Merged

Revert "[CI][Deps] Uplift CPU/FPGAEMU RT version to 2023.15.3.0.20" #8952

merged 1 commit into from
Apr 5, 2023

Conversation

yanfeng3721
Copy link
Contributor

Reverts #8851
It looks like we are missing some thing which cause CPURT failed to be detected. Revert this PR first to unblock the massive failure in pre-commit for llvm-test-suite tests.

@yanfeng3721 yanfeng3721 requested a review from a team as a code owner April 5, 2023 14:04
@dm-vodopyanov
Copy link
Contributor

Looks like this is not a culprit for llvm-test-suite failure, because this patch updates OpenCL CPU and FPGA Fast Emu RTs, but in that failure GPU is affected as well.

@yanfeng3721
Copy link
Contributor Author

yanfeng3721 commented Apr 5, 2023 •
edited
Loading

Looks like this is not a culprit for llvm-test-suite failure, because this patch updates OpenCL CPU and FPGA Fast Emu RTs, but in that failure GPU is affected as well.

Yes, just trigger this revert PR to verify the suspicious guilty commit, please do not merge the PR if the "no devices (OpenCL CPU, OpenCL/L0 GPU)" issue on pre Commit testing persist. Thanks.

@yanfeng3721 yanfeng3721 temporarily deployed to aws April 5, 2023 14:32 — with GitHub Actions Inactive
@yanfeng3721
Copy link
Contributor Author

Seems revert PR can resolve the "no devices (OpenCL CPU, OpenCL/L0 GPU)" issue, please help to merge the PR as soon as all pre commit test pass, thanks.

sycl-ls --verbose
  [opencl:cpu:0] Intel(R) OpenCL, AMD Ryzen 7 5800X 8-Core Processor              OpenCL 3.0 (Build 0) [2022.14.8.0.04_160000]

@yanfeng3721 yanfeng3721 temporarily deployed to aws April 5, 2023 15:23 — with GitHub Actions Inactive
dm-vodopyanov
dm-vodopyanov approved these changes Apr 5, 2023
View reviewed changes
Copy link
Contributor

@dm-vodopyanov dm-vodopyanov left a comment

Choose a reason for hiding this comment

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

That's really strange.

@dm-vodopyanov dm-vodopyanov merged commit 305da17 into sycl Apr 5, 2023
@bader bader deleted the revert-8851-uplift_ocl_cpu_fpgaemu_rt branch April 5, 2023 16:29
yanfeng3721 added a commit that referenced this pull request Apr 13, 2023
@yanfeng3721
Revert "Revert "[CI][Deps] Uplift CPU/FPGAEMU RT version to 2023.15.3…
b5769d3 
….0.20" (#8952)"

This reverts commit 305da17.
@yanfeng3721 yanfeng3721 mentioned this pull request Apr 13, 2023
Merged
stdale-intel pushed a commit that referenced this pull request Apr 24, 2023
@yanfeng3721
Revert "Revert "[CI][Deps] Uplift CPU/FPGAEMU RT version to 2023.15.3…
0ce9b5f 
….0.20"" (#9059)

Reverts #8952
Try resubmit #8851
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@aelovikov-intel aelovikov-intel aelovikov-intel approved these changes

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
@yanfeng3721 @dm-vodopyanov @aelovikov-intel