[PATCH] D37118: [ARM] Tidy-up ARMAsmParser. NFC.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 07:00:35 PDT 2017


asb added a comment.

Why not use MCRegisterInfo::getSubReg for this purpose, thus avoiding repeating information that's already present in ARMRegisterInfo.td? `MRI->getSubReg(QReg, ARM::dsub_0)` should replace the functionality of getDRegFromQReg.


https://reviews.llvm.org/D37118





More information about the llvm-commits mailing list