[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