Skip to content

Conversation

@zhaomaosu
Copy link
Contributor

If user used all SLM, enqueue kernel will failed with out of device memory. We'd better report more detailed messages then user will not confuse about this error.

If user used all SLM, enqueue kernel will failed with out of device
memory. We'd better report more detailed messages then user will not
confuse about this error.
@zhaomaosu zhaomaosu requested a review from a team as a code owner September 9, 2025 07:25
@zhaomaosu zhaomaosu requested a review from a team September 9, 2025 07:25
Copy link
Contributor

@kswiecicki kswiecicki left a comment

Choose a reason for hiding this comment

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

Small nits, otherwise LGTM.

@zhaomaosu
Copy link
Contributor Author

Hi @intel/llvm-gatekeepers, this PR is ready to be merged. Thanks.

@steffenlarsen steffenlarsen merged commit c0c92a8 into intel:sycl Sep 12, 2025
65 of 67 checks passed
@zhaomaosu zhaomaosu deleted the asan-report-error-detail branch September 12, 2025 05:57
AlexeySachkov pushed a commit to AlexeySachkov/llvm that referenced this pull request Sep 29, 2025
…l#20010)

If user used all SLM, enqueue kernel will failed with out of device
memory. We'd better report more detailed messages then user will not
confuse about this error.
AlexeySachkov added a commit that referenced this pull request Oct 23, 2025
This is a cherry-pick of #20010

If user used all SLM, enqueue kernel will failed with out of device
memory. We'd better report more detailed messages then user will not
confuse about this error.

Patch-by: Maosu Zhao <[email protected]>
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.

4 participants