[llvm] [SPIRV] Emit intrinsics for globals only in function that references them (PR #178143)

[Konrad Kleine via llvm-commits]

Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?=,
Juan Manuel Martinez =?utf-8?q?Caamaño?Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/178143 at github.com>


kwk wrote:

> > Reverting this due to a suspicious ASAN failure: #179268
> 
> It seems that this PR may not be the root cause of the failure. But the failure is there over main on a reduced example (and this PR triggered it).
> 
> I'll work on fixing this and then reapply this PR wihtout modificaitons.
> 
> ```assembly
> ; with ASAN
> ; llc -verify-machineinstrs -O0 -mtriple=spirv32-unknown-unknown reduced.ll -o /dev/null --spirv-ext=+SPV_INTEL_function_pointers
> define void @foo() {
> entry:
>   store ptr addrspace(4) addrspacecast (ptr @foo to ptr addrspace(4)), ptr null, align 8
>   ret void
> }
> ```

@jmmartinez were you able to identify or fix the ASAN issue? [Apparently](https://github.com/llvm/llvm-project/pull/179268#issuecomment-3836499074) you can reproduce the issue without this PR.

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