[PATCH] D62739: AMDGPU: Always emit amdgpu-flat-work-group-size

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 31 10:33:01 PDT 2019


arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: lib/CodeGen/TargetInfo.cpp:7885
+    // By default, restrict the maximum size to 256.
+    F->addFnAttr("amdgpu-flat-work-group-size", "128,256");
   }
----------------
b-sumner wrote:
> Theoretically, shouldn't the minimum be 1?
That's what I thought, but the backend is defaulting to 2 * wave size now


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62739/new/

https://reviews.llvm.org/D62739





More information about the cfe-commits mailing list