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

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 11:33:00 PDT 2016


tstellarAMD added a comment.

I think this looks OK overall.  I'm a little nervous about the changes to scratch handling introducing bugs, since this is a little hard to test.  I would recommend doing a shader-db run on this code, because that has more register spills than piglit.


http://reviews.llvm.org/D18340





More information about the llvm-commits mailing list