[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