Skip to content

Conversation

@kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Feb 16, 2023

The new ur_program_properties_t struct allows the provision of
additional metadata to be passed into the
urProgramCreate/urProgramCreateWithBinary entry points.
Additionally, it supports stype and pNext which enable extending
program creation without breaking ABI.

Fixes #95

@kbenzie kbenzie force-pushed the benie/program-metadata branch from d5daec5 to 3b923d8 Compare February 17, 2023 11:15
Copy link
Contributor

@veselypeta veselypeta left a comment

Choose a reason for hiding this comment

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

LGTM

@kbenzie kbenzie force-pushed the benie/program-metadata branch from 3b923d8 to 967c56d Compare February 17, 2023 13:21
@kbenzie kbenzie requested a review from aarongreig February 17, 2023 13:48
@kbenzie kbenzie force-pushed the benie/program-metadata branch from 967c56d to f9927a0 Compare February 20, 2023 11:35
Copy link
Contributor

@aarongreig aarongreig left a comment

Choose a reason for hiding this comment

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

LGTM

The new `ur_program_properties_t` struct allows the provision of
additional metadata to be passed into the
`urProgramCreate`/`urProgramCreateWithBinary` entry points.
Additionally, it supports `stype` and `pNext` which enable extending
program creation without breaking ABI.

Fixes oneapi-src#95
@kbenzie kbenzie force-pushed the benie/program-metadata branch from f9927a0 to f1732b5 Compare February 21, 2023 16:53
@kbenzie kbenzie merged commit 495ae3a into oneapi-src:main Feb 21, 2023
@kbenzie kbenzie deleted the benie/program-metadata branch February 21, 2023 17:10
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.

Reconsider urProgramCreateWithBinary signature

6 participants