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

[Carl Ritson via Phabricator via llvm-commits]

critson 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 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.
@foad your suggested test change seems fine to me


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86578