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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 5 15:47:45 PST 2017


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM



================
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
----------------
You should be able to just remove this whole block when the triple setting is removed


https://reviews.llvm.org/D39643





More information about the llvm-commits mailing list