[PATCH] D86578: [TargetLowering] Combine known bits for icmp in SimplifySetCC (PR41182)

Tue Jan 18 09:32:36 PST 2022

RKSimon added inline comments.

================
Comment at: llvm/test/CodeGen/AMDGPU/wave32.ll:720
+; GFX1032: s_and_saveexec_b32 [[SAVE1:s[0-9]+]], [[SAVE2]]
+; GFX1032: s_or_saveexec_b32 [[SAVE2]], -1
 ; GFX1032: s_mov_b32 exec_lo, [[SAVE2]]
----------------
foad wrote:
> RKSimon wrote:
> > @foad I guess there's no good way around this if on wave32 targets - unless we duplicate the test with a 16 divergent variant?
> Let's just replace the "cmp uge 32" with "cmp uge 16", and then your patch won't affect this test at all. (@critson is that OK?) Would you like me to do that in a separate commit?
Not sure if the commit question was to me or @critson - but if you can alter the test and then ping this ticket I will rebase the patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86578