[PATCH] D69654: AMDGPU: Switch backend default max workgroup size to 1024

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 31 11:07:27 PDT 2019


arsenm added a comment.

In D69654#1729022 <https://reviews.llvm.org/D69654#1729022>, @rampitec wrote:

> I do not think that deliberately introducing performance regression is a good way to force FE to do anything.


clang already emits the clamp to 256 if unspecified. The bugs from not being correct by default have come up many times


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

https://reviews.llvm.org/D69654





More information about the llvm-commits mailing list