[PATCH] IAS: support .rep as an alias for .rept

Renato Golin renato.golin at linaro.org
Wed Dec 25 11:07:55 PST 2013


  Apart from my two comments, looks good to me.


================
Comment at: lib/MC/MCParser/AsmParser.cpp:3967
@@ -3959,3 +3966,3 @@
   if (parseAbsoluteExpression(Count))
-    return TokError("unexpected token in '.rept' directive");
+    return TokError("unexpected token in '" + Dir + "' directive");
 
----------------
I see why you're passing DK here, but I think the message could just as well be '.rep/.rept' and avoid the extra argument when there is no need for the logic, just error message.

TokError will report the line, right?

================
Comment at: test/MC/AsmParser/directive_rept.s:1
@@ +1,2 @@
+# RUN: llvm-mc -triple i686-elf -filetype asm -o - %s | FileCheck %s
+
----------------
this might need to be in an x86-specific section... ARM bots don't compile the x86 back-end and this will likely fail.


http://llvm-reviews.chandlerc.com/D2457



More information about the llvm-commits mailing list