[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