[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
Wed Feb 1 09:34:54 PST 2023
goldstein.w.n added a comment.
@nikic, I think I may have added too many tests. The issue is there are alot of edge cases around the `nsw`/`nuw` flags that I felt needed to be tested. If you see any that you think are redundant, however, am happy to drop.
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