[Lldb-commits] [PATCH] D70574: [lldb-server] Verify 'g' is supported before relying on them

Jason Molenda via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 21 16:01:38 PST 2019

jasonmolenda added a comment.

Hi, sorry I haven't had time to get back to this one.  I wanted to look into the m_avoid_g_packets ivar in GDBRemoteCommunicationClient and GDBRemoteCommunicationClient::AvoidGPackets.  It looks like it was written to specifically avoid the g packet with arm64 devices, and I think it's trying to avoid the g packet for older debugservers.

The last time we enabled this, the macos bots immediately started getting failures -- I suspect there is a bug with debugserver's g packet right now, which we hadn't noticed because we don't use the g packet on this platform (we pre-seed all of the GPR values so it's not a perf problem).  I need to spend a little time looking into what's going on there --- but we'll probably need to make sure GDBRemoteCommunicationClient::AvoidGPackets is called / used, and that it is updated to avoid some more debugservers. :)

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list