[llvm] [AMDGPU] Introduce "amdgpu-sw-lower-lds" pass to lower LDS accesses. (PR #87265)

via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 07:47:55 PDT 2024


skc7 wrote:

> Alright, this might be useful for sanitizer workloads but it looks broken to me.
> 
> Please put a loop in the start of the pass that checks whether the module has any sanitizer things in it (function attribute iirc?) and immediate returns without changing the IR at all if there are no uses of the sanitizer. Provided it is clear that this does not execute on any non-sanitizer IR I'm happy to hit the green button.

Have recently merged a PR [#99439 ](https://github.com/llvm/llvm-project/pull/99439) which adds   "nosanitize_address" module flag when asan pass is run. Updated this PR to use the same flag to return early if no sanitizer instrumentation is found.

https://github.com/llvm/llvm-project/pull/87265


More information about the llvm-commits mailing list