[llvm] [AMDGPU] Merge two V_CNDMASK instructions into V_DUAL_CNDMASK (PR #135007)
    Jay Foad via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Apr  9 07:50:49 PDT 2025
    
    
  
jayfoad wrote:
> > > for using vop2 instead of vop3 format
> > 
> > 
> > Normally this kind of shrinking would be done in SIShrinkInstructions. Is there a reason you can't do it there?
> 
> I wasn't sure if that would be proper because I am changing not only v_cndmask but also cmp instructions, but correct me if i'm wrong
I think it is OK to do it in SIShrinkInstructions. It does not have stricter rules than SIFoldOperands.
https://github.com/llvm/llvm-project/pull/135007
    
    
More information about the llvm-commits
mailing list