[PATCH] D56320: [mips] Optimize shifts for types larger than GPR size (mips2/mips3)
Simon Atanasyan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 9 13:29:12 PST 2019
atanasyan accepted this revision.
atanasyan added a comment.
This revision is now accepted and ready to land.
LGTM with a couple of nits
================
Comment at: lib/Target/Mips/MipsISelLowering.cpp:2443
+
+ if (!(Subtarget.hasMips4() || Subtarget.hasMips32())) {
+ SDVTList VTList = DAG.getVTList(VT, VT);
----------------
clang-format the `if` block please
================
Comment at: lib/Target/Mips/MipsISelLowering.cpp:4380
+ // have the same condition operand. On machines which don't have
+ // conditional-move instrution, it reduces unnecessary branch instructions
+ // which are result of using two diamond patterns that are result of two
----------------
s/instrution/instruction/
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56320/new/
https://reviews.llvm.org/D56320
More information about the llvm-commits
mailing list