Skip to content

Mirror intel/llvm commits #2794

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 3 commits into from
Jun 24, 2025
Merged

Mirror intel/llvm commits #2794

merged 3 commits into from
Jun 24, 2025

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jun 24, 2025

Automated changes by create-pull-request GitHub action

kbenzie and others added 3 commits June 24, 2025 00:41
This query isn't accessible publicly, removing it has not caused any
test failures.
and fix context lifetime management in command_list_manager. There were
two issues:
- move ctor and assignment operators were declared as default but that
was incorrect - they should set context on the moved-from
command_list_manager to nullptr to avoid double free.
- urContextRelease() was called in the desturctor, which means its
context could have been released before other members. This was a
problem since zeCommandList was trying to return command_list to the
context's cache in it's dtor.
@kbenzie kbenzie requested a review from a team as a code owner June 24, 2025 00:41
Copy link
Contributor

Unified Runtime -> intel/llvm Repo Move Notice

Information

The source code of Unified Runtime has been moved to intel/llvm under the unified-runtime top-level directory,
all future development will now be carried out there. This was done in intel/llvm#17043.

The code will be mirrored to oneapi-src/unified-runtime and the specification will continue to be hosted at oneapi-src.github.io/unified-runtime.

The contribution guide will be updated with new instructions for contributing to Unified Runtime.

PR Migration

All open PRs including this one will be marked with the auto-close label and shall be automatically closed after 30 days.

Should you wish to continue with your PR you will need to migrate it to intel/llvm.
We have provided a script to help automate this process.

If your PR should remain open and not be closed automatically, you can remove the auto-close label.


This is an automated comment.

@aarongreig aarongreig merged commit 36376e0 into main Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants