[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.

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

Merged

martygrant merged 45 commits into sycl from review/yang/restructure_asan_msan

Dec 11, 2024

Contributor

AllanZyne commented Nov 1, 2024 •

edited

Loading

UR: oneapi-src/unified-runtime#2325

AllanZyne added 3 commits

October 23, 2024 19:35


          restructure

07b6361


          update ur

1daaed7


          Merge branch 'sycl' into review/yang/restructure_asan

51d3f84

AllanZyne requested review from a team as code owners

November 1, 2024 03:29

AllanZyne requested a review from bso-intel

November 1, 2024 03:29

AllanZyne marked this pull request as draft

November 1, 2024 03:29

zhaomaosu had a problem deploying to WindowsCILock

November 18, 2024 08:07

— with

GitHub Actions Error

AllanZyne had a problem deploying to WindowsCILock

November 18, 2024 08:51

— with

GitHub Actions Failure

AllanZyne had a problem deploying to WindowsCILock

November 18, 2024 10:07

— with

GitHub Actions Failure

AllanZyne had a problem deploying to WindowsCILock

November 19, 2024 02:58

— with

GitHub Actions Error

zhaomaosu had a problem deploying to WindowsCILock

November 19, 2024 03:34

— with

GitHub Actions Error

zhaomaosu had a problem deploying to WindowsCILock

November 19, 2024 03:58

— with

GitHub Actions Failure

zhaomaosu had a problem deploying to WindowsCILock

November 19, 2024 04:46

— with

GitHub Actions Failure

zhaomaosu reviewed

View reviewed changes

llvm/lib/SYCLLowerIR/ComputeModuleRuntimeInfo.cpp Outdated Show resolved Hide resolved

zhaomaosu reviewed

View reviewed changes

libdevice/include/msan_rtl.hpp Outdated Show resolved Hide resolved

zhaomaosu reviewed

View reviewed changes

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp Outdated Show resolved Hide resolved

zhaomaosu reviewed

View reviewed changes

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp Outdated Show resolved Hide resolved

zhaomaosu reviewed

View reviewed changes

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp Outdated Show resolved Hide resolved

zhaomaosu reviewed

View reviewed changes

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp Outdated Show resolved Hide resolved

AllanZyne added 7 commits

November 20, 2024 00:28


          Merge branch 'sycl' into review/yang/restructure_asan

ee3ba02


          sync with msan poc

f291d3f


          fix build

245d5ae


          fix driver test

1b74d0f


          add comment

5507bf0


          Merge branch 'sycl' into review/yang/restructure_asan


          libsycl-asan skip amd and nv

221175c

AllanZyne changed the base branch from sycl to review/yang/restructure_asan

November 21, 2024 05:59

mdtoguchi approved these changes

View reviewed changes

AllanZyne added 2 commits

December 9, 2024 21:58


          fix win

6b1084c


          Merge branch 'sycl' into review/yang/restructure_asan_msan

a1f8358

AllanZyne had a problem deploying to WindowsCILock

December 10, 2024 06:02

— with

GitHub Actions Error


          fix driver lit test

7037a82

AllanZyne had a problem deploying to WindowsCILock

December 10, 2024 06:08

— with

GitHub Actions Failure

AllanZyne had a problem deploying to WindowsCILock

December 10, 2024 07:02

— with

GitHub Actions Failure


          fix windows

b0d60a2

AllanZyne temporarily deployed to WindowsCILock

December 10, 2024 07:30

— with

GitHub Actions Inactive

AllanZyne temporarily deployed to WindowsCILock

December 10, 2024 08:43

— with

GitHub Actions Inactive

AlexeySachkov approved these changes

View reviewed changes


          Merge branch 'sycl' into review/yang/restructure_asan_msan

8174b9c

AllanZyne had a problem deploying to WindowsCILock

December 11, 2024 02:45

— with

GitHub Actions Failure


          fix build

f189391

AllanZyne temporarily deployed to WindowsCILock

December 11, 2024 03:09

— with

GitHub Actions Inactive

AllanZyne temporarily deployed to WindowsCILock

December 11, 2024 04:05

— with

GitHub Actions Inactive

kbenzie added 2 commits

December 11, 2024 11:08


          [UR] Bump main tag to 8818ab5

bb0326d


          Merge remote-tracking branch 'origin/sycl' into review/yang/restructu…

c20acc4

…re_asan_msan

kbenzie had a problem deploying to WindowsCILock

December 11, 2024 11:11

— with

GitHub Actions Failure

kbenzie temporarily deployed to WindowsCILock

December 11, 2024 12:26

— with

GitHub Actions Inactive

kbenzie temporarily deployed to WindowsCILock

December 11, 2024 13:26

— with

GitHub Actions Inactive

kbenzie approved these changes

View reviewed changes

kbenzie temporarily deployed to WindowsCILock

December 11, 2024 14:16

— with

GitHub Actions Inactive

Contributor

kbenzie commented Dec 11, 2024

@intel/llvm-gatekeepers please merge

martygrant merged commit cbd6b7c into sycl

14 checks passed

AllanZyne deleted the review/yang/restructure_asan_msan branch

December 12, 2024 01:11

KornevNikita pushed a commit that referenced this pull request


          [DeviceMSAN] Support MemorySanitizer for device offloading (#15955)

9b2b854

UR: oneapi-src/unified-runtime#2325

---------

Co-authored-by: Zhao, Maosu <[email protected]>
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

Reviewers

yingcong-wu yingcong-wu left review comments

AlexeySachkov AlexeySachkov approved these changes

zhaomaosu zhaomaosu approved these changes

mdtoguchi mdtoguchi approved these changes

steffenlarsen steffenlarsen approved these changes

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

+1 more reviewer

kbenzie kbenzie approved these changes

Labels

None yet