Skip to content

[SYCL][Driver] Enable SPV_INTEL_memory_access_aliasing extension #4025

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[SYCL][Driver] Enable SPV_INTEL_memory_access_aliasing extension #4025

wants to merge 1 commit into from

Conversation

MrSidims
Copy link
Contributor

Signed-off-by: Dmitry Sidorov [email protected]

@MrSidims MrSidims requested a review from vzakhari June 29, 2021 17:50
@MrSidims
Copy link
Contributor Author

/summary:run

@MrSidims
Copy link
Contributor Author

InvalidModule: Invalid SPIR-V module: input SPIR-V module uses unknown extension 'SPV_INTEL_memory_access_aliasing'

Moved to draft.

@MrSidims MrSidims marked this pull request as draft June 29, 2021 18:33
vzakhari
vzakhari previously approved these changes Jun 30, 2021
View reviewed changes
Copy link
Contributor

@vzakhari vzakhari left a comment

Choose a reason for hiding this comment

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

LGTM

@MrSidims MrSidims marked this pull request as ready for review July 7, 2021 16:43
@MrSidims
Copy link
Contributor Author

MrSidims commented Jul 7, 2021

I believe with GPU RT update the patch is ready to be tested and reviewed.

@MrSidims MrSidims force-pushed the private/MrSidims/MemAccessSPV branch from 241a34c to f0577ea Compare August 10, 2021 08:09
@MrSidims MrSidims requested a review from hchilama as a code owner August 10, 2021 08:09
@MrSidims MrSidims force-pushed the private/MrSidims/MemAccessSPV branch from f0577ea to 6730cd2 Compare August 10, 2021 08:14
@MrSidims
Copy link
Contributor Author

/summary:run

@MrSidims MrSidims marked this pull request as draft August 18, 2021 08:36
@MrSidims MrSidims closed this Oct 26, 2021
@MrSidims MrSidims reopened this Oct 26, 2021
@MrSidims MrSidims force-pushed the private/MrSidims/MemAccessSPV branch 3 times, most recently from 310f376 to e89f1bd Compare October 26, 2021 10:25
@MrSidims
Copy link
Contributor Author

/summary:run

@MrSidims MrSidims force-pushed the private/MrSidims/MemAccessSPV branch from e89f1bd to 2e4fe27 Compare December 8, 2021 12:24
@MrSidims
Copy link
Contributor Author

MrSidims commented Dec 8, 2021

/summary:run

@MrSidims MrSidims marked this pull request as ready for review January 17, 2022 02:24
@MrSidims MrSidims requested a review from a team as a code owner January 17, 2022 02:24
@MrSidims MrSidims force-pushed the private/MrSidims/MemAccessSPV branch from 2e4fe27 to 79d19e7 Compare January 17, 2022 02:26
@MrSidims
Copy link
Contributor Author

/summary:run

@github-actions github-actions bot added the Stale label Jul 17, 2022
@github-actions github-actions bot closed this Aug 16, 2022
@MrSidims MrSidims reopened this Aug 23, 2022
@MrSidims MrSidims mentioned this pull request Sep 22, 2022
Closed
@github-actions github-actions bot closed this Sep 23, 2022
@airMeng airMeng mentioned this pull request Jul 18, 2024
Closed
@maarquitos14 maarquitos14 mentioned this pull request Aug 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AGindinson AGindinson Awaiting requested review from AGindinson

@mdtoguchi mdtoguchi Awaiting requested review from mdtoguchi

@hchilama hchilama Awaiting requested review from hchilama

1 more reviewer

@vzakhari vzakhari vzakhari left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MrSidims @vzakhari