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

Tue Jan 18 02:28:52 PST 2022

foad added a subscriber: critson.
foad 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]]
----------------
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?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86578