[PATCH] D18340: AMDGPU: allow specifying a workgroup size that needs to fit in a compute unit

[Matt Arsenault via llvm-commits]

arsenm added a comment.

I have another patch which already fixes a hardcoded maximum of 256: http://reviews.llvm.org/D16834

You can try merging this what that one.

New attributes should be lowercased,use  '-' separated words and start with an "amdgpu-" prefix


================
Comment at: lib/Target/AMDGPU/AMDGPU.h:131-132
@@ -130,3 +130,4 @@
     GEOMETRY = 2,
-    COMPUTE = 3
+    COMPUTE = 3,
+    GL_COMPUTE = 4
   };
----------------
I think theses should be distinguished by the ABI rather than introducing a new shader type. Also semi-related, I would like to replace shader types with calling conventions


http://reviews.llvm.org/D18340