Skip to content

[DeviceASAN] Fix throw "UR_RESULT_ERROR_INVALID_ARGUMENT" exception when catching free related error #2592

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

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

AllanZyne
Copy link
Contributor

@AllanZyne AllanZyne commented Jan 21, 2025

LLVM: intel/llvm#16706

a small enhancement

@AllanZyne
Copy link
Contributor Author

@oneapi-src/unified-runtime-maintain, please review. Thanks

@pbalcer pbalcer added the ready to merge Added to PR's which are ready to merge label Feb 6, 2025
@AllanZyne
Copy link
Contributor Author

Hi @oneapi-src/unified-runtime-maintain, please merge. Thanks.

@kbenzie kbenzie merged commit 7e38d0a into oneapi-src:main Feb 13, 2025
75 checks passed
@kbenzie
Copy link
Contributor

kbenzie commented Feb 14, 2025

Unfortunately due to the test failures on intel/llvm#16706 I'll need to revert this change to unblock the merge queue.

@AllanZyne AllanZyne deleted the review/yang/fix_asan_free branch February 17, 2025 06:09
kbenzie added a commit to intel/llvm that referenced this pull request Feb 20, 2025
…hen catching free related error (#16706)

UR: oneapi-src/unified-runtime#2592

---------

Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request Feb 21, 2025
…free related error (#16706)

UR: oneapi-src#2592

---------

Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants