Skip to content

Conversation

@nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Oct 7, 2023

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.

Requires related patch in Unified Runtime Adapters here: oneapi-src/unified-runtime#934

@nrspruit nrspruit changed the title [UR][L0] Add support for passing device list to urProgramBuild/Link/C… [UR][L0] Add support for passing device list to urProgramBuild/Link/Compile Oct 7, 2023
…ompile

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.

Requires related patch in Unified Runtime Adapters here: oneapi-src/unified-runtime#934

Signed-off-by: Spruit, Neil R <neil.r.spruit@intel.com>
@nrspruit nrspruit force-pushed the fixprogrambuild_updated branch from 59733a6 to 55a872f Compare November 17, 2023 19:47
@kbenzie
Copy link
Contributor

kbenzie commented Nov 21, 2023

@nrspruit oneapi-src/unified-runtime#934 has been merged so I've updated the UR tag and merged in the latest sycl branch changes.

@kbenzie kbenzie marked this pull request as ready for review November 21, 2023 11:41
@kbenzie kbenzie requested a review from a team as a code owner November 21, 2023 11:41
@kbenzie
Copy link
Contributor

kbenzie commented Nov 21, 2023

I think I know why the checks are failing. Working on a fix.

@kbenzie
Copy link
Contributor

kbenzie commented Nov 22, 2023

@intel/llvm-gatekeepers please merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants