[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