8000 Mirror intel/llvm commits by kbenzie · Pull Request #2776 · oneapi-src/unified-runtime · GitHub
[go: up one dir, main page]

Skip to content

Mirror intel/llvm commits #2776

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

Mirror intel/llvm commits #2776

merged 4 commits into from
Jun 3, 2025

Conversation

kbenzie
Copy link
Contributor
@kbenzie kbenzie commented Jun 3, 2025

Automated changes by create-pull-request GitHub action

kbenzie and others added 4 commits June 3, 2025 00:42
All global DDI table entries were in fact adapter entries. This patch
moves these entries into the adapter class thus removing the
`urGetGlobalProcAddrTable()` entry.
Some Vulkan interop tests did not correctly check for the
`ext-oneapi-bindless-images` aspect. This lead to failures when devices
(e.g. iGPU) supported memory import, but did not support Bindless
Images.

Adding the `// REQUIRES` lit directive fixed this issue for iGPU,
however, it caused problems for Intel BMG machines, which do not pass
the aspect query as it is currently implemented.

To fix this, the query now returns Bindless Image support based on
extensions returned from `zeDriverGetExtensionProperties`.

The `env` vars set by LIT in the `buffer_usm.cpp` test are unnecessary
and have been removed. This was causing a problem on iGPU where the
interop would usually work, but the test was failing due to the `env`
variables set for Bindless Images.

A number of improvements have been made to `vulkan_common.hpp`. There is
now a clearer distinction between required and optional instance and
device extensions. Optional extensions are loaded only when supported.
Fetching function pointers for optional extensions is now protected so
that devices with no support do not try to attempt to retrieve pointers
to those functions.

The `VK_IMAGE_USAGE_STORAGE_BIT` was removed from image creation as it
was not necessary to the functionality of the tests.

The size of the depth texture created in `depth_format.cpp` was
increased due to larger minimum memory allocation size requirements on
iGPU.
We have two exp variants of urEnqueueKernelLaunch that don't differ
significantly from the core entry point, and both have been stable for
over a year. This change merges these entry points into the core one,
and also pulls all related exp functionality into the core spec.

The exp features in question are exp-cooperative-kernels and
exp-launch-properties.
@kbenzie kbenzie requested a review from a team as a code owner June 3, 2025 00:42
@github-actions github-actions bot added ci/cd Continuous integration/devliery loader Loader related feature/bug conformance Conformance test suite issues. specification Changes or additions to the specification experimental Experimental feature additions/changes/specification level-zero L0 adapter specific issues cuda CUDA adapter specific issues hip HIP adapter specific issues opencl OpenCL adapter specific issues native-cpu Native CPU adapter specific issues labels Jun 3, 2025
Copy link
Contributor
github-actions bot commented Jun 3, 2025

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.

@kbenzie kbenzie merged commit 990c7d4 into main Jun 3, 2025
1 check passed
@kbenzie kbenzie deleted the mirror-commits- branch June 3, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-close ci/cd Continuous integration/devliery conformance Conformance test suite issues. cuda CUDA adapter specific issues experimental Experimental feature additions/changes/specification hip HIP adapter specific issues level-zero L0 adapter specific issues loader Loader related feature/bug native-cpu Native CPU adapter specific issues opencl OpenCL adapter specific issues specification Changes or additions to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0