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

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 07:55:19 PDT 2016


compnerd added a subscriber: compnerd.
compnerd accepted this revision.
compnerd added a reviewer: compnerd.
compnerd added a comment.
This revision is now accepted and ready to land.

Nice!


================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:12109
@@ +12108,3 @@
+  if (Subtarget->hasDivide()) {
+    unsigned DivOpcode = isSigned ? ISD::SDIV : ISD::UDIV;
+    const SDValue Dividend = Op->getOperand(0);
----------------
Why not inline this into the `Div`?


https://reviews.llvm.org/D24133





More information about the llvm-commits mailing list