[Lldb-commits] [lldb] [lldb][AArch64] Add SME2's ZT0 register (PR #70205)

Jason Molenda via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 31 15:15:02 PDT 2023


================
@@ -488,6 +508,12 @@ bool RegisterInfoPOSIX_arm64::IsSMERegZA(unsigned reg) const {
   return reg == m_sme_regnum_collection[2];
 }
 
+bool RegisterInfoPOSIX_arm64::IsSMERegZT(unsigned reg) const {
+  // ZT0 is part of the SME register set only if SME2 is present.
+  return m_sme_regnum_collection.size() == 4 &&
+         reg == m_sme_regnum_collection.back();
----------------
jasonmolenda wrote:

Do you think it might be better to check if the register number is contained in `m_sme_rengum_collection` instead of expecting there to be 4 registers and this is the last one?  It seems like this is could be a bug when we get another register in the next SME update in the future.

https://github.com/llvm/llvm-project/pull/70205


More information about the lldb-commits mailing list