[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