[PATCH] D29887: [mips] divide macro instruction cleanup.

Sagar Thakur via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 13 07:13:33 PST 2017


slthakur accepted this revision.
slthakur added a comment.
This revision is now accepted and ready to land.

LGTM with minor nits.



================
Comment at: lib/Target/Mips/AsmParser/MipsAsmParser.cpp:3416
+//
+// Notably we don't have emit a warning when encountering $rt is the $zero
+// register, or 0 as an immediate. processInstruction() has already done that.
----------------
Should this be like -> "Notably we don't have **to** emit a warning when encountering $rt **as** the $zero register"?


================
Comment at: lib/Target/Mips/AsmParser/MipsAsmParser.cpp:3419
+//
+// The destination register should $zero register should only be seen in
+// the immediate case.
----------------
Could you please rephrase this statement so that it is clear to understand?


https://reviews.llvm.org/D29887





More information about the llvm-commits mailing list