[PATCH] ARM unwind directive parser, assembler, and testcases.

[Logan Chien]

  Thanks for your review.


================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:8034
@@ +8033,3 @@
+        Parser.getTok().isNot(AsmToken::Dollar)) {
+      return Error(Parser.getTok().getLoc(), "'#' expected");
+    }
----------------
Rafael Ávila de Espíndola wrote:
> '#' or '$'?
IMHO, it will be better to use "'#' expect" in this patch.  Some existing code is using this diagnostic message as well (such as parsePKHImm(), parseShifterImm(), parseRotImm(), and etc.)  Besides, it seems that the dollar sign was added for backward compatibility with legacy code.  Maybe it will be better to suggest the programmer to use the new prefix for constant literals.


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

BRANCH
  add-arm-unwind-directives-squashed

ARCANIST PROJECT
  llvm