[PATCH] D87976: Support the division-by-constant strength reduction for more integer types

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

I'm skeptical this is a good idea when the division is wider than the widest legal mulhi.  You end up generating either a ton of inline code, or a libcall; the result might not be faster than the original divide libcall.

The improvements along the lines of llvm/test/CodeGen/AArch64/srem-seteq.ll are obviously profitable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87976