[PATCH] [mips][microMIPS] Implement CodeGen support for SW16 and LW16 instructions
Sasa Stankovic
Sasa.Stankovic at imgtec.com
Wed Feb 4 02:10:12 PST 2015
LGTM, with additional comment.
================
Comment at: lib/Target/Mips/MipsSEISelDAGToDAG.cpp:427
@@ +426,3 @@
+
+ if (selectAddrRegImm(Addr, Base, Offset))
+ return false;
----------------
I think it would be good if you add a comment for this if statement, because it behaves differently than the other similar "if (select...)" statements in this file (it returns false when the select function returns true, while the other similar statements return true when the select function returns true). The comment might be (for example):
For all the other cases where "lw" would be selected, don't select "lw16".
http://reviews.llvm.org/D6581
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list