Skip to content
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

xe: remove ngen relative directories #2860

Merged
merged 1 commit into from
Mar 12, 2025
Merged

xe: remove ngen relative directories #2860

merged 1 commit into from
Mar 12, 2025

Conversation

rjoursler
Copy link
Contributor

@rjoursler rjoursler commented Mar 12, 2025 •
edited
Loading

For third party builds, the relative directory will resolve to the wrong path.

Implementation of proposal in #2853 for merging.

@rjoursler
xe: remove ngen relative directories
4717582 
For third party builds, the relative directory will resolve to the wrong path.
@rjoursler rjoursler requested a review from a team as a code owner March 12, 2025 00:21
@github-actions github-actions bot added platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel backport labels Mar 12, 2025
@rjoursler rjoursler mentioned this pull request Mar 12, 2025
Merged
@rjoursler
Copy link
Contributor Author

make test
disable test_device_cpu
enable test_device_gpu

@rjoursler rjoursler merged commit b75b206 into rls-v3.8-pc Mar 12, 2025
11 of 14 checks passed
@rjoursler rjoursler deleted the rjoursle/third_party_ngen_conflict branch March 12, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vpirogov vpirogov vpirogov approved these changes

@kealan-barbieri kealan-barbieri kealan-barbieri approved these changes

Assignees
No one assigned
Labels
backport platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rjoursler @vpirogov @kealan-barbieri