[PATCH] D100904: [AMDGPU] Lower regbanks reassign threshold to 15000

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 02:05:49 PDT 2021


rampitec added a comment.

In D100904#2707688 <https://reviews.llvm.org/D100904#2707688>, @foad wrote:

> In D100904#2705174 <https://reviews.llvm.org/D100904#2705174>, @rampitec wrote:
>
>> In D100904#2704460 <https://reviews.llvm.org/D100904#2704460>, @foad wrote:
>>
>>> Is there a measurable benefit on any kernels?
>>
>> Not that I know.
>
> Then maybe delete the pass, or at least disable it by default?

Both are options as well as to enable only at -O3. However, I still think we have not enough data. JBTW, what gfx benchmark suite tells us?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100904



More information about the llvm-commits mailing list