[PATCH] D24133: [ARM] Lower UDIV+UREM to UDIV+MLS (and the same for SREM)

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 13:14:43 PDT 2016


rengolin added a comment.

In https://reviews.llvm.org/D24133#531896, @jmolloy wrote:

> Also, we can avoid linking in uidivmod at all, which in small programs is *significant* in terms of code size reduction.


That's true.

Division with all the speed optimisations and "code maintenance improvements" is a *very* large algorithm, and can incur into calls between themselves to save copy&paste. Just look at our compiler-rt implementation... :)


https://reviews.llvm.org/D24133





More information about the llvm-commits mailing list