Skip to content
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

[DeviceMSAN][Draft] Memory Sanitizer POC #15955

Draft
wants to merge 11 commits into
base: sycl
Choose a base branch
from
Draft

[DeviceMSAN][Draft] Memory Sanitizer POC #15955

wants to merge 11 commits into from

Conversation

AllanZyne
Copy link
Contributor

@AllanZyne AllanZyne commented Nov 1, 2024 •
edited
Loading

@AllanZyne AllanZyne requested review from a team as code owners November 1, 2024 03:29
@AllanZyne AllanZyne marked this pull request as draft November 1, 2024 03:29
zhaomaosu
zhaomaosu reviewed Nov 20, 2024
View reviewed changes
llvm/lib/SYCLLowerIR/ComputeModuleRuntimeInfo.cpp
@@ -58,6 +58,19 @@ bool isModuleUsingAsan(const Module &M) {
return false;
}

bool isModuleUsingMsan(const Module &M) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this function if we don't use it now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we need this function, I changed its checking algorithm

@AllanZyne AllanZyne changed the base branch from sycl to review/yang/restructure_asan November 21, 2024 05:59
@AllanZyne AllanZyne force-pushed the review/yang/restructure_asan_msan branch from 8d3db4c to 5ede3e9 Compare November 21, 2024 06:47
Base automatically changed from review/yang/restructure_asan to sycl November 23, 2024 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhaomaosu zhaomaosu zhaomaosu left review comments

@bso-intel bso-intel Awaiting requested review from bso-intel bso-intel is a code owner automatically assigned from intel/llvm-reviewers-runtime

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AllanZyne @zhaomaosu