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

Fri Jun 14 08:27:57 PDT 2019

arsenm added a comment.

In D62739#1543578 <https://reviews.llvm.org/D62739#1543578>, @b-sumner wrote:

> In D62739#1543438 <https://reviews.llvm.org/D62739#1543438>, @arsenm wrote:
>
> > In D62739#1536428 <https://reviews.llvm.org/D62739#1536428>, @b-sumner wrote:
> >
> > > We need to communicate with anyone generating IR to ensure this is being generated before we change the default.  clang is only one of those generators.  This change will also need to be documented in the usage document.
> >
> >
> > The planned change is to make the backend more conservative, so it shouldn't break other frontends
>
>
> It may not break other frontends, but could cause substantial performance regressions.  At a minimum the summary should clearly mention this possibility.

The summary of the future backend patch will

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

https://reviews.llvm.org/D62739