[PATCH] D115173: [InstCombine] try to fold rem with constant dividend and select-of-constants divisor

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 7 11:44:20 PST 2021


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Could you please add this test: https://godbolt.org/z/WvYxnejnc (it should not optimize to unreachable/undef/poison)

LGTM, but this reminds me of D71561 <https://reviews.llvm.org/D71561> that got stuck (sorry, that was not my intention),
similar generalization could be applied here i believe.


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

https://reviews.llvm.org/D115173



More information about the llvm-commits mailing list