[PATCH] D48610: [ARM] Add missing Thumb2 assembler diagnostics.

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 27 01:00:22 PDT 2018


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

Looks OK to me.



================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:6405
   case ARM::STRD: {
-    // Rt2 must be Rt + 1.
     unsigned Rt = MRI->getEncodingValue(Inst.getOperand(0).getReg());
     unsigned Rt2 = MRI->getEncodingValue(Inst.getOperand(1).getReg());
----------------
Nit: all these cases are all so similar, and differ only in a few constants. While you're at it, do you think these patterns can be factored out? 


Repository:
  rL LLVM

https://reviews.llvm.org/D48610





More information about the llvm-commits mailing list