Skip to content

Cherry-pick multi device compile spec changes#1083

Closed
aarongreig wants to merge 3 commits intooneapi-src:adaptersfrom
aarongreig:aaron/cherrypickMultiDeviceCompileSpec
Closed

Cherry-pick multi device compile spec changes#1083
aarongreig wants to merge 3 commits intooneapi-src:adaptersfrom
aarongreig:aaron/cherrypickMultiDeviceCompileSpec

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: jaime.a.arteaga.molina@intel.com
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