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

Pablo Barrio via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 6 04:03:24 PDT 2016


pbarrio updated this revision to Diff 70377.
pbarrio added a comment.

Improved testing:

- New signed remainder test.
- New sdiv + srem test.
- Registers are given as regular expressions.

All tests merge to udiv/sdiv + mls.

I am not sure about adding a div+div+mod test, though. I have checked that
two identical divs + remainder merge just fine. However, is that an IR that
we can typically get? I would expect the two divs to be merged before getting
to the back-end.


https://reviews.llvm.org/D24133

Files:
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/urem-opt-size.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24133.70377.patch
Type: text/x-patch
Size: 3913 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160906/f13a3fe3/attachment-0001.bin>


More information about the llvm-commits mailing list