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.

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

[L0] Fix external semaphore import function calls to match the header #2467

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Dec 13, 2024

  • Fix the function pointer for the external semaphore import function calls to match the updated header.

@nrspruit nrspruit requested review from a team as code owners December 13, 2024 19:29
@nrspruit nrspruit added the v0.11.x Include in the v0.11.x release label Dec 13, 2024
@github-actions github-actions bot added images UR images level-zero L0 adapter specific issues labels Dec 13, 2024
@nrspruit nrspruit force-pushed the fix_external_import_function_call branch from cb57571 to 65933ee Compare December 13, 2024 19:38
nrspruit added a commit to nrspruit/llvm that referenced this pull request Dec 13, 2024
-pre-commit PR for oneapi-src/unified-runtime#2467

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
- Fix the function pointers for the external semaphore import function
  calls to match the updated header.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit force-pushed the fix_external_import_function_call branch from 65933ee to be27d8f Compare December 13, 2024 19:53
nrspruit added a commit to nrspruit/llvm that referenced this pull request Dec 13, 2024
-pre-commit PR for oneapi-src/unified-runtime#2467

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit changed the title [L0] Fix external semaphore import function call to match the header [L0] Fix external semaphore import function calls to match the header Dec 13, 2024
@nrspruit nrspruit added the ready to merge Added to PR's which are ready to merge label Dec 13, 2024
@nrspruit
Copy link
Contributor Author

@oneapi-src/unified-runtime-bindless-images-write , please approve since this fixes mistake due to errors in the extensions documentation in the driver. The docs have been fixed alongside the fix in the adapter.

@kbenzie kbenzie merged commit 39df031 into oneapi-src:main Dec 16, 2024
73 checks passed
kbenzie added a commit that referenced this pull request Dec 16, 2024
[L0] Fix external semaphore import function calls to match the header
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
images UR images level-zero L0 adapter specific issues ready to merge Added to PR's which are ready to merge v0.11.x Include in the v0.11.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants