[PATCH] D110065: [AArch64] Add support for the 'R' architecture profile.

John Brawn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 25 05:56:05 PDT 2021


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

I have one small comment, but otherwise LGTM.



================
Comment at: llvm/lib/Target/AArch64/MCTargetDesc/AArch64InstPrinter.cpp:1558
+
+  if (Reg && !isValidSysReg(Reg, Read, STI))
+    Reg = AArch64SysReg::lookupSysRegByName(Reg->AltName);
----------------
It would be good to have a comment here explaining what's going on.


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

https://reviews.llvm.org/D110065



More information about the cfe-commits mailing list