Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[XPTI] Use ur.call rather than ur in XPTI #1940

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

RossBrunton
Copy link
Contributor

Closes: #14922

@RossBrunton RossBrunton requested a review from a team as a code owner August 6, 2024 13:27
@github-actions github-actions bot added loader Loader related feature/bug specification Changes or additions to the specification labels Aug 6, 2024
@RossBrunton
Copy link
Contributor Author

Requires DPCPP changes from intel/llvm#14971

@RossBrunton RossBrunton force-pushed the ross/urcall branch 4 times, most recently from 7e460d5 to 3fe361c Compare August 13, 2024 11:16
@RossBrunton RossBrunton added the ready to merge Added to PR's which are ready to merge label Aug 14, 2024
@RossBrunton
Copy link
Contributor Author

Should be ready to merge now - the failing E2E test is because there are required intel/llvm changes.

@RossBrunton RossBrunton force-pushed the ross/urcall branch 2 times, most recently from e575c03 to 69d6804 Compare August 16, 2024 13:37
@RossBrunton RossBrunton added the v0.10.x Include in the v0.10.x release label Aug 20, 2024
@omarahmed1111 omarahmed1111 merged commit d0a5052 into oneapi-src:main Aug 20, 2024
69 of 72 checks passed
kbenzie pushed a commit that referenced this pull request Aug 22, 2024
[XPTI] Use `ur.call` rather than `ur` in XPTI
@kbenzie kbenzie mentioned this pull request Aug 22, 2024
53 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loader Loader related feature/bug ready to merge Added to PR's which are ready to merge specification Changes or additions to the specification v0.10.x Include in the v0.10.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants