[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