Skip to content

Update device sanitizer instrument of static workgroup local memory per change in https://github.com/intel/llvm/pull/16356 #16979

@wenju-he

Description

@wenju-he

Describe the bug

Currently AddressSanitizerPass is checking __sycl_allocateLocalMemory call and then instrument static local memory.
Since #16356 the call is resolved to global variable before AddressSanitizerPass.
Therefore, AddressSanitizerPass should be updated to instrument the global variable instead.

Test https://github.com/intel/llvm/blob/sycl/sycl/test-e2e/AddressSanitizer/out-of-bounds/local/group_local_memory.cpp will fail after #16356 because local memory isn't instrumented.
So I'm disabling the test in #16959. The test should be re-enabled when this issue is fixed.

To reproduce

Environment

Additional context

No response

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions