[PATCH] D36692: [ARM, Asm] Add diagnostics for general-purpose register operands

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 11:54:51 PDT 2017


rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

Other than the one comment, LGTM. Thanks!



================
Comment at: test/MC/ARM/ldrd-strd-gnu-thumb-bad-regs.s:3
+
+@ FIXME: These errors are inaccurate because the error is being reported on the
+@ implicit r13 operand added after r12.
----------------
Right, this is possibly worse than before. Do you have a patch in the queue?


Repository:
  rL LLVM

https://reviews.llvm.org/D36692





More information about the llvm-commits mailing list