Skip to content

Conversation

@veselypeta
Copy link
Contributor

@veselypeta veselypeta commented Apr 11, 2023

Hold-off merging this until intel/llvm#8763 is merged - in case there are additional changes.
Upstream PR has now been merged

Closes #427

@veselypeta veselypeta force-pushed the petr/427/add-backend-option branch from 7398f74 to 61255d7 Compare April 11, 2023 09:44
Copy link
Contributor

@kbenzie kbenzie left a comment

Choose a reason for hiding this comment

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

LGTM, one suggestion

@veselypeta veselypeta force-pushed the petr/427/add-backend-option branch from 61255d7 to a94961a Compare April 11, 2023 14:15
@veselypeta veselypeta force-pushed the petr/427/add-backend-option branch from a94961a to 0f3d083 Compare April 12, 2023 08:03
@veselypeta veselypeta force-pushed the petr/427/add-backend-option branch from 74b2abe to 6687f35 Compare April 12, 2023 08:27
Copy link
Contributor

@callumfare callumfare left a comment

Choose a reason for hiding this comment

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

LGTM

@veselypeta veselypeta merged commit 4f48ed8 into oneapi-src:main Apr 14, 2023
@veselypeta veselypeta deleted the petr/427/add-backend-option branch April 14, 2023 09:00
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.

Add entry-point to UR which matches piPluginGetBackendOptions

5 participants