Skip to content

Cherry-pick multi device compile spec changes #1083

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

Conversation

aarongreig
Copy link
Contributor

This should allow us to merge and upstream #934

igchor and others added 2 commits November 15, 2023 15:42
piProgramBuild receives a list of devices, while urProgramBuild
does not. This produces a series of issues when a UR program
needs to be created for a specific device.

So define a new API, called urProgramBuildExp to pass this
list.

Authored-by: [email protected]
Expand upon the introduction of `urProgramBuildExp` and include
`urProgramCompileExp` and `urProgramLinkExp` which include a device-list
in place of a context. These more closely align with the PI/OpenCL
analogues but only to introduce device-lists, not all extant arguments
from those entry-points. This patch also moves the `urProgramBuildExp`
definition into an experimental feature file and introduces a brief
document containing motivation.
@kbenzie
Copy link
Contributor

kbenzie commented Nov 15, 2023

ur_adapter_level_zero.def : error LNK2001: unresolved external symbol urGetProgramExpProcAddrTable [D:\a\unified-runtime\unified-runtime\build\source\adapters\level_zero\ur_adapter_level_zero.vcxproj]

We'll need to add urGetProgramExpProcAddrTable to the adapters.

@aarongreig aarongreig requested review from a team as code owners November 15, 2023 16:31
@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Nov 16, 2023
@kbenzie
Copy link
Contributor

kbenzie commented Nov 16, 2023

I've pulled this change into #934 so we don't need a separate PR.

@kbenzie kbenzie closed this Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Added to PR's which are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants