[PATCH] D142021: AMDGPU/GlobalISel: Add stub custom regbankselect pass
    Matt Arsenault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jan 19 11:07:27 PST 2023
    
    
  
arsenm added a comment.
In D142021#4064779 <https://reviews.llvm.org/D142021#4064779>, @rovka wrote:
> Why do we need a new pass? Could we achieve the same goals with a new RegBankSelect mode instead? (Sorry if this was discussed somewhere else and I missed it, in any case I think the commit message should either explain the rationale or point to wherever it's been discussed)
I believe the mode is an orthogonal choice, which is part of why I am trying to subclass the existing pass. We could have the same use and def analysis like other targets with the different modes, but the uniformity driven decisions are more fundamental. Additionally this needs to be a separate pass because it introduces new required analysis passes
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142021/new/
https://reviews.llvm.org/D142021
    
    
More information about the llvm-commits
mailing list