[PATCH] D143014: [InstCombine] Add combines for `(urem/srem (mul/shl X, Y), (mul/shl X, Z))`

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 09:52:44 PST 2023


goldstein.w.n added a comment.

In D143014#4107118 <https://reviews.llvm.org/D143014#4107118>, @MattDevereau wrote:

> I think it would also be beneficial to split the Const and Generic cases into separate patches to simplify the review process

Done, see: D143417 <https://reviews.llvm.org/D143417> for the generic, this patch now only handles the constant case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143014



More information about the llvm-commits mailing list