[clang] [llvm] [llvm][opt][Transforms][SPIR-V] Enable `InferAddressSpaces` for SPIR-V (PR #110897)

Alex Voicu via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 11 16:59:38 PDT 2024


https://github.com/AlexVlx commented:

> Move to separate change, not sure this is necessarily valid for spirv  

I think that I'd prefer to keep this around, definitely for AMDGCNSPIRV where we know it is both correct and empirically beneficial. For vanilla SPIR-V I'll defer to folks on that side - I cannot think about cases where it'd be legal to put anything but a pointer to global (CrossWorkgroup) in constant (UniformConstant), but that might simply be ignorance on my part.

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


More information about the cfe-commits mailing list