Skip to content

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jan 22, 2025

…-on-shadow-per-type

[DeivceASAN] Make ShadowMemory one instance per type
…nel_release

[DeviceASAN] Fix kernel release order
@github-actions github-actions bot added the loader Loader related feature/bug label Jan 22, 2025
@kbenzie
Copy link
Contributor Author

kbenzie commented Jan 22, 2025

@yingcong-wu @AllanZyne there were merge conflicts due to us having merged #1536 to main branch. Please check this to make sure I've resolved those correctly.

@kbenzie kbenzie marked this pull request as ready for review January 22, 2025 16:18
@kbenzie kbenzie requested a review from a team as a code owner January 22, 2025 16:18
Copy link
Contributor

@AllanZyne AllanZyne left a comment

Choose a reason for hiding this comment

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

Looks good for #2586

@yingcong-wu
Copy link
Contributor

yingcong-wu commented Jan 23, 2025

The failed checks are expected as there is an API change with urKernelSuggestMaxCooperativeGroupCountExp, and the API is not changed for v0.11.

See in

Copy link
Contributor

@yingcong-wu yingcong-wu left a comment

Choose a reason for hiding this comment

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

Please wait for #2605. This is a bugfix for #2585.

@kbenzie
Copy link
Contributor Author

kbenzie commented Jan 23, 2025

Please wait for #2605. This is a bugfix for #2585.

Okay, I'll merge this to UR main & intel/llvm then cherry-pick it onto this branch.

@yingcong-wu
Copy link
Contributor

Please wait for #2605. This is a bugfix for #2585.

Okay, I'll merge this to UR main & intel/llvm then cherry-pick it onto this branch.

Thank you!

@kbenzie kbenzie merged commit 688179d into oneapi-src:v0.11.x Jan 23, 2025
71 of 73 checks passed
@kbenzie kbenzie deleted the v0.11.5rc branch January 23, 2025 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loader Loader related feature/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants