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

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 02:14:46 PDT 2016


rengolin added a comment.

Makes sense. The i64 type is "legal" because of the long divmod library call, which is not legal for UDIV/SDIV.

Shouldn't i16 and i8 also work, here? I'm guessing that they have type promotion earlier on, so just adding some tests should be fine.

If not, than you may need to update the type check (and maybe extend some types manually).

cheers,
--renato


https://reviews.llvm.org/D25077





More information about the llvm-commits mailing list