[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