Skip to content

[CI] Bump IGC version #16635

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
Jan 15, 2025
Merged

[CI] Bump IGC version #16635

merged 1 commit into from
Jan 15, 2025

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Jan 14, 2025

We are currently using 24.52.32224.5 and the corresponding IGC version is v.2.5.6 as per here. The script doesn't update it, I'll fix it tomorrow but for now just manually bump it.

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex requested a review from bader January 14, 2025 21:36
@sarnex
Copy link
Contributor Author

sarnex commented Jan 14, 2025

Some tests are expected to XPASS, I'll fix them in a new commit tomorrow if it happens.

@sarnex sarnex marked this pull request as ready for review January 14, 2025 21:41
@sarnex sarnex requested a review from a team as a code owner January 14, 2025 21:41
@bader
Copy link
Contributor

bader commented Jan 14, 2025

Some tests are expected to XPASS, I'll fix them in a new commit tomorrow if it happens.

If would be great if you can re-enable sycl/test-e2e/Regression/local-arg-align.cpp as I do in https://github.com/intel/llvm/pull/13783/files. If not, I'll check myself when IGC is updated.

@bader
Copy link
Contributor

bader commented Jan 14, 2025

Some tests are expected to XPASS, I'll fix them in a new commit tomorrow if it happens.

If would be great if you can re-enable sycl/test-e2e/Regression/local-arg-align.cpp as I do in https://github.com/intel/llvm/pull/13783/files. If not, I'll check myself when IGC is updated.

@sarnex, sorry, just realized that we should keep this test disabled on Windows platform. We don't have the fixed Windows driver yet.

@sarnex
Copy link
Contributor Author

sarnex commented Jan 14, 2025

Sure, I'll wait until we have the CI results before changing any tests.

@sarnex sarnex merged commit ed1b6fd into intel:sycl Jan 15, 2025
27 checks passed
@sarnex
Copy link
Contributor Author

sarnex commented Jan 15, 2025

@bader Seems everything passed (no XPASS), so I'm not sure what's going on with the local-arg-align test.

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