[PATCH] D54546: [ARM] Don't expand sdiv when optimising for minsize

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 13:42:33 PST 2018


efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:7820
+
+  auto SDIV = MinSize && HasDivide ? SDValue(N, 0) : SDValue();
+
----------------
Please early return here instead of changing the value of SDIV.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54546/new/

https://reviews.llvm.org/D54546





More information about the llvm-commits mailing list