[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