-
Notifications
You must be signed in to change notification settings - Fork 769
[DeviceMSAN] Support MemorySanitizer for device offloading #15955
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
Changes from 25 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
07b6361
restructure
AllanZyne 1daaed7
update ur
AllanZyne 51d3f84
Merge branch 'sycl' into review/yang/restructure_asan
AllanZyne ee3ba02
Merge branch 'sycl' into review/yang/restructure_asan
AllanZyne f291d3f
sync with msan poc
AllanZyne 245d5ae
fix build
AllanZyne 1b74d0f
fix driver test
AllanZyne 5507bf0
add comment
AllanZyne 7711096
Merge branch 'sycl' into review/yang/restructure_asan
AllanZyne 221175c
libsycl-asan skip amd and nv
AllanZyne 5ede3e9
address comments
AllanZyne 91f7390
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne ad809b4
fix crash
AllanZyne ac08ed7
Add memory buffer test
zhaomaosu a707baa
add driver tests
AllanZyne 7384106
update sycl-post-link test
AllanZyne eefc9ea
Add __MsanKernelMetadata
AllanZyne a1b7cf7
add sycl-post-link test
AllanZyne 850cedd
add msan test
AllanZyne 5f77720
fix driver test
AllanZyne 23049c4
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne cb7de24
disable gen12
AllanZyne 4e60de2
fix post link
AllanZyne a3384cf
disable win driver test
AllanZyne 51547db
rename MemoryMapParams
AllanZyne fb32ace
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne 7452e5d
merge sanitizer kernel metadata pass
AllanZyne 3e9b9ad
fix asan tests
AllanZyne efad229
driver: add warning on windows
AllanZyne 38e1a8e
address comments
AllanZyne 05ec5f4
fix getOrCreateGlobalString
AllanZyne faf9b59
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne a7eed82
fix window build
AllanZyne 7a884f0
fix win build
AllanZyne 572dafd
add xfail and xfail-tracker
AllanZyne 28e635b
add driver test on windows
AllanZyne 44400b7
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne 6b1084c
fix win
AllanZyne a1f8358
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne 7037a82
fix driver lit test
AllanZyne b0d60a2
fix windows
AllanZyne 8174b9c
Merge branch 'sycl' into review/yang/restructure_asan_msan
AllanZyne f189391
fix build
AllanZyne bb0326d
[UR] Bump main tag to 8818ab5b
kbenzie c20acc4
Merge remote-tracking branch 'origin/sycl' into review/yang/restructu…
kbenzie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the name is different for ASAN and MSAN?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. I will merge them in other patch.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently, I handle them all in SanitizerKernelMetadata Pass, and don't use a same name, because I'm not sure if their content will be same in the future.