llvm [DeviceSanitizer] Support detecting misaligned access error

UR: https://github.com/oneapi-src/unified-runtime/pull/1747

Jun 12 '24 03:06 AllanZyne

@intel/llvm-reviewers-runtime @intel/unified-runtime-reviewers please review. Thank you very much!

Jun 21 '24 08:06 AllanZyne

The failed test "no_sycl_hpp_in_e2e_tests.cpp" is not related to this PR.

Jul 02 '24 06:07 AllanZyne

The failed test "no_sycl_hpp_in_e2e_tests.cpp" is not related to this PR.

Hi @AllanZyne ! It is related. The test makes sure that no E2E tests include <sycl/sycl.hpp>. Instead, the new tests should use <sycl/detail/core.hpp> and similar headers (I suspect you'll also need <sycl/usm.hpp>.)

Jul 02 '24 06:07 steffenlarsen

The failed test "no_sycl_hpp_in_e2e_tests.cpp" is not related to this PR.

Hi @AllanZyne ! It is related. The test makes sure that no E2E tests include <sycl/sycl.hpp>. Instead, the new tests should use <sycl/detail/core.hpp> and similar headers (I suspect you'll also need <sycl/usm.hpp>.)

Sorry for not checking this. New patch pushed.

Jul 02 '24 06:07 AllanZyne

@intel/unified-runtime-reviewers please review. Thanks.

Jul 04 '24 06:07 AllanZyne

@intel/unified-runtime-reviewers please review. Thank you very much!

Jul 09 '24 11:07 AllanZyne

@intel/unified-runtime-reviewers please review. Thank you very much!

I've added the UR PR to the ready to merge queue. Will get to it soon.

Jul 10 '24 09:07 kbenzie

@intel/llvm-gatekeepers please merge

Jul 11 '24 19:07 kbenzie