[Lldb-commits] [PATCH] D62931: [lldb-server] Add setting to force 'g' packet use

Jason Molenda via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 28 17:21:39 PDT 2019


jasonmolenda added a comment.

In D62931#1724433 <https://reviews.llvm.org/D62931#1724433>, @guiandrade wrote:

> Yeah, more tests would be useful. They made me notice an issue btw. I was simply sending a 'g' packet and checking if the server replied back nicely; however, IIUC with 'G' packets it isn't so simple because we also need to send the registers data in the same packet. I'm not sure how I could do that, and I'm also afraid that check could get too expensive. Do you have any idea what could be a nice way to handle that?


fwiw I can't imagine a stub that supports g but not G.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62931/new/

https://reviews.llvm.org/D62931





More information about the lldb-commits mailing list