Skip to content

Conversation

jandres742
Copy link

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.

@jandres742 jandres742 force-pushed the multiDevProgramBuild branch 2 times, most recently from 90abeed to 4bb5d3f Compare October 1, 2023 01:38
@jandres742 jandres742 changed the title Add support for passing device list to urProgramBuild [UR][L0] Add support for passing device list to urProgramBuild Oct 1, 2023
@jandres742
Copy link
Author

intel/llvm pre-merge testing in intel/llvm#11371

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.

Signed-off-by: Jaime Arteaga <[email protected]>
Copy link
Contributor

@pbalcer pbalcer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Jamie, it seems you have modified the API files directly. You need to modify the spec yaml files instead. The process is outlined here.

@@ -87,6 +87,7 @@ UR_DLLEXPORT ur_result_t UR_APICALL urGetProgramProcAddrTable(
return result;
}
pDdiTable->pfnBuild = urProgramBuild;
pDdiTable->pfnBuildExp = urProgramBuildExp;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pDdiTable->pfnBuildExp = urProgramBuildExp;
pDdiTable->pfnBuildExp = nullptr;

@kbenzie
Copy link
Contributor

kbenzie commented Oct 3, 2023

Superceed by #919 based on the v0.7.x branch, #924 based on main, and the multi-device-compile-hotfix branch based on 3c6f02c.

@kbenzie kbenzie closed this Oct 3, 2023
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.

3 participants