[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