[PATCH] D106079: [AMDGPU] Divergence-driven compare operations instruction selection

[Carl Ritson via Phabricator via llvm-commits]

critson added a comment.

Can you explain the motivation for this change further?
Particularly as it increases generated code size.



================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:4475
+  case AMDGPU::S_CMP_EQ_U64: return AMDGPU::V_CMP_EQ_U64_e64;
+  case AMDGPU::S_CMP_LG_U64: return AMDGPU::V_CMP_NE_U64_e64;
   case AMDGPU::S_BCNT1_I32_B32: return AMDGPU::V_BCNT_U32_B32_e64;
----------------
Do we now have to always use larger e64 instructions or are these later reduced if we can use vcc instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106079