[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