================
@@ -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
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to