[PATCH] D88785: Support {S,U}REMEqFold before legalization

Simonas Kazlauskas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 18 15:37:10 PDT 2021


nagisa added a comment.

@RKSimon I cherry-picked your patch on top of mine and there were no changes in test output (as reported by `check-llvm`), sadly. If anything your differential makes improvements by this differential slightly less impressive.

I think that makes sense, though, because `{S,U}REM-SetEq` folds down to a regular multiplication, and not `MULHU`/`MULHS`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88785



More information about the llvm-commits mailing list