[PATCH] D39643: [AMDGPU] Fix assertion due to assuming pointer in default addr space is 32 bit

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 5 18:19:45 PST 2017


yaxunl added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:2136
 
   if (GlobalAddressSDNode *GA = dyn_cast<GlobalAddressSDNode>(Callee)) {
+    // FIXME: Remove this hack for function pointer types after removing
----------------
arsenm wrote:
> You should be able to just remove this whole block when the triple setting is removed
Yes I will do that.


https://reviews.llvm.org/D39643





More information about the llvm-commits mailing list