[Lldb-commits] [lldb] [lldb][AArch64][Linux] Add support for SME only systems (PR #165413)

[Omair Javaid via lldb-commits]

================
@@ -1598,38 +1752,55 @@ void NativeRegisterContextLinux_arm64::ConfigureRegisterContext() {
   // If m_sve_state is set to SVEState::Disabled on first stop, code below will
   // be deemed non operational for the lifetime of current process.
   if (!m_sve_header_is_valid && m_sve_state != SVEState::Disabled) {
-    // If we have SVE we may also have the SVE streaming mode that SME added.
-    // We can read the header of either mode, but only the active mode will
-    // have valid register data.
+    // Systems may have SVE and/or SME. If they are SME only, the SVE regset
+    // cannot be read from but the SME one can. If they have both SVE and SME,
+    // only the active mode will return valid register data.
 
-    // Check whether SME is present and the streaming SVE mode is active.
     m_sve_header_is_valid = false;
----------------
omjavaid wrote:

Comment here was removed. Lets add a comment here describing SVEState::Streaming means SME and we are checking for SME.



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