[PATCH] D24714: [MC] Support skip and count for .incbin directive

Petr Hosek via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 20 19:46:15 PDT 2016


phosek added a comment.

That would be awesome, they're not really related to this one but I'd like to land them as well.


================
Comment at: lib/MC/MCParser/AsmParser.cpp:4407-4411
@@ +4406,7 @@
+  if (getLexer().isNot(AsmToken::EndOfStatement)) {
+    if (parseToken(AsmToken::Comma,
+                   "unexpected token in '.incbin' directive") ||
+        parseAbsoluteExpression(Skip))
+      return true;
+ 
+    if (getLexer().isNot(AsmToken::EndOfStatement)) {
----------------
davide wrote:
> this and the lines at 4413-4417 are identical. Can you factor out in a common helper?
This no longer relevant as these blocks are now different.


Repository:
  rL LLVM

https://reviews.llvm.org/D24714





More information about the llvm-commits mailing list