[PATCH] D104773: [Aarch64] Adding support for Armv9-A Realm Management Extension

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64SystemOperands.td:478
 defm : TLBI<"VAALE1",       0b000, 0b1000, 0b0111, 0b111>;
+defm : TLBI<"RPAOS",        0b110, 0b1000, 0b0100, 0b011>;
+defm : TLBI<"RPALOS",       0b110, 0b1000, 0b0100, 0b111>;
----------------
Should these be in a different Requires block, for RME?


================
Comment at: llvm/lib/Target/AArch64/AArch64SystemOperands.td:751
+// v9a Realm Management Extension registers
+def : RWSysReg<"MFAR_EL3",  0b11, 0b110, 0b0110, 0b0000, 0b101>;
+def : RWSysReg<"GPCCR_EL3", 0b11, 0b110, 0b0010, 0b0001, 0b110>;
----------------
Is this register writable?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104773