Address Omair's comments

Author: DavidSpickett

lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp

@@ -2802,10 +2802,11 @@ GDBRemoteCommunicationServerLLGS::Handle_qMemoryRegionInfo(
     if (memory_tagged != MemoryRegionInfo::eDontKnow ||
         is_shadow_stack != MemoryRegionInfo::eDontKnow) {
       response.PutCString("flags:");
+      // Space is the separator.
       if (memory_tagged == MemoryRegionInfo::eYes)
-        response.PutCString("mt");
+        response.PutCString("mt ");
       if (is_shadow_stack == MemoryRegionInfo::eYes)
-        response.PutCString("ss");
+        response.PutCString("ss ");
 
       response.PutChar(';');
     }

lldb/unittests/Process/Utility/LinuxProcMapsTest.cpp

@@ -292,6 +292,21 @@ INSTANTIATE_TEST_SUITE_P(
                     MemoryRegionInfo::eDontKnow, MemoryRegionInfo::eDontKnow,
                     MemoryRegionInfo::eDontKnow),
             },
+            ""),
+        // We must look for exact flag strings, ignoring substrings of longer
+        // flag names.
+        std::make_tuple(
+            "0-0 rw-p 00000000 00:00 0\n"
+            "VmFlags: amt mtb amtb",
+            MemoryRegionInfos{
+                MemoryRegionInfo(
+                    make_range(0, 0), MemoryRegionInfo::eYes,
+                    MemoryRegionInfo::eYes, MemoryRegionInfo::eNo,
+                    MemoryRegionInfo::eNo, MemoryRegionInfo::eYes,
+                    ConstString(nullptr), MemoryRegionInfo::eDontKnow, 0,
+                    MemoryRegionInfo::eNo, MemoryRegionInfo::eDontKnow,
+                    MemoryRegionInfo::eNo),
+            },
             "")));
 
 TEST_P(LinuxProcSMapsTestFixture, ParseSMapRegions) {