[PATCH] D16774: AMDGPU: Whitelist handled intrinsics

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 08:39:11 PST 2016

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


Comment at: lib/Target/AMDGPU/AMDGPUPromoteAlloca.cpp:546-551
@@ -522,3 +545,8 @@
     IntrinsicInst *Intr = dyn_cast<IntrinsicInst>(Call);
     if (!Intr) {
+      // FIXME: What is this for? It doesn't make sense to promote arbitrary
+      // function calls. If the call is to a defined function that can also be
+      // promoted, we should be able to do this once that function is also
+      // rewritten.
       std::vector<Type*> ArgTypes;
I'm not sure why this is here.  If it doesn't make sense, feel free to remove it.


More information about the llvm-commits mailing list