[PATCH] [mips] [IAS] Add support for the .insn directive.

Daniel Sanders daniel.sanders at imgtec.com
Thu Apr 2 02:33:57 PDT 2015


LGTM with a small change.


================
Comment at: lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp:39-40
@@ +38,4 @@
+    markLabelsAsMicroMips();
+  else
+    Labels.clear();
+}
----------------
We should unconditionally clear Labels as the previous code did. We don't need to mark the same labels multiple times.

http://reviews.llvm.org/D8006

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list