[PATCH] D73176: [ARM] Fix dropped dollar sign from symbols in branch targets

Lucas Prates via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 24 06:22:54 PST 2020


pratlucas added a comment.

I agree with resolving the ambiguity in favor of identifiers regardless of the mnemonic for the general-case operand parsing, avoiding inconsistencies. Specific cases where constant immediates are explicitly expected are currently handled by their own ParseMethod implementation.
If there's no objection, I'll update the patch to reflect this behavior.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73176/new/

https://reviews.llvm.org/D73176





More information about the llvm-commits mailing list