Skip to content

[DeviceASAN] Use device usm to sync asan runtime data instead of shared usm #2249

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 8 commits into from
Dec 5, 2024

Conversation

zhaomaosu
Copy link
Contributor

@zhaomaosu zhaomaosu commented Oct 28, 2024

Shared USM has poor performance, change it to device USM will benefit several benchmarks.

intel/llvm part: intel/llvm#15911

…ed usm

Shared USM has poor performance, change it to device USM will benefit
several benchmarks.
@zhaomaosu zhaomaosu requested a review from a team as a code owner October 28, 2024 03:15
@github-actions github-actions bot added loader Loader related feature/bug sanitizer Sanitizer layer issues/changes/specification labels Oct 28, 2024
@pbalcer
Copy link
Contributor

pbalcer commented Oct 28, 2024

this is a non-trivial change, please create an intel/llvm PR.

@zhaomaosu
Copy link
Contributor Author

this is a non-trivial change, please create an intel/llvm PR.

Done

@kbenzie kbenzie added the v0.11.x Include in the v0.11.x release label Nov 25, 2024
@AllanZyne
Copy link
Contributor

please rebase.

@pbalcer
Copy link
Contributor

pbalcer commented Dec 4, 2024

This needs a rebase again.

@zhaomaosu
Copy link
Contributor Author

This needs a rebase again.

Done

@zhaomaosu
Copy link
Contributor Author

The build failure is fixed in intel/llvm part PR. And the pre ci of intel/llvm PR is passed.

@pbalcer pbalcer added the ready to merge Added to PR's which are ready to merge label Dec 5, 2024
@martygrant martygrant merged commit a172cde into oneapi-src:main Dec 5, 2024
72 of 74 checks passed
martygrant added a commit to intel/llvm that referenced this pull request Dec 5, 2024
KornevNikita pushed a commit to intel/llvm that referenced this pull request Feb 19, 2025
KornevNikita pushed a commit to intel/llvm that referenced this pull request Feb 25, 2025
KornevNikita pushed a commit to intel/llvm that referenced this pull request Feb 25, 2025
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 sanitizer Sanitizer layer issues/changes/specification v0.11.x Include in the v0.11.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants