[llvm] [AMDGPU] Merge two V_CNDMASK instructions into V_DUAL_CNDMASK (PR #135007)

Ana Mihajlovic via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 9 08:51:01 PDT 2025


mihajlovicana 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.

okay, will try and migrate that there 

https://github.com/llvm/llvm-project/pull/135007


More information about the llvm-commits mailing list