[PATCH] D135900: [AMDGPU][MC][GFX8+] Correct v_cndmask modifiers

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 12:09:56 PDT 2022


Joe_Nash added a comment.

I guess CNDMASK_B16 should be a True16 instruction. I shall plan to do that. With that in mind, this patch appears even less parameterized for (i32 vs i16) input types than before. Is there any way to get some of that back? IE before this patch CNDMASK_B16_T16 wouldn't need to override InsDPP, but after this it does.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135900/new/

https://reviews.llvm.org/D135900



More information about the llvm-commits mailing list