[PATCH] D43281: [AMDGPU] fixes for lds f32 builtins
Daniil Fukalov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 1 09:56:31 PST 2018
dfukalov added a comment.
The problem is that if set addrspace "2" in description string, CanT.getAddressSpace() returns target addrspace value "11" (shifted in the enum) and compares it with input LangAS addrspace ("2", "opencl_local" in our case).
So I cannot set a number a description string that will be equal to LangAS addrspace "opencl_local".
Moreover, this change is preparation to move to custom processing of these builtins. Then I'm going to remove link (GCCBuiltin in IntrinsicsAMDGPU.td) from the llvm intrinsics definitions. Then I'll be able to switch on custom processing in cfe.
https://reviews.llvm.org/D43281
More information about the llvm-commits
mailing list