[PATCH] D57608: [NFC][GlobalISel]: Add a convenience method to MachineInstrBuilder to simplify getOperand(i).getReg()
    Matt Arsenault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Feb  2 07:52:00 PST 2019
    
    
  
arsenm added a comment.
In D57608#1382034 <https://reviews.llvm.org/D57608#1382034>, @kparzysz wrote:
> In D57608#1381719 <https://reviews.llvm.org/D57608#1381719>, @aemerson wrote:
>
> > If we're going to add a helper to shave some characters off, we might as well go for something really short like getOpReg().
>
>
> Or simply `getReg(unsigned OpIdx)`---this would keep the names identical.  Also, should there be something like this for subregs as well?
No, subregisters should never exist for G_* instructions
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57608/new/
https://reviews.llvm.org/D57608
    
    
More information about the llvm-commits
mailing list