[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