labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
Looks good. Please pay special attention to the windows bot when landing. ================ Comment at: lldb/source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp:55-63 OnRunPacketSent(true); - // The main ReadPacket loop wakes up at computed_timeout intervals, just to + // The main ReadPacket loop wakes up at computed_timeout intervals, just to // check that the connection hasn't dropped. When we wake up we also check // whether there is an interrupt request that has reached its endpoint. - // If we want a shorter interrupt timeout that kWakeupInterval, we need to + // If we want a shorter interrupt timeout that kWakeupInterval, we need to // choose the shorter interval for the wake up as well. + std::chrono::seconds computed_timeout = ---------------- I think this could be simplified now (to avoid the intermittent wakeups), but those wakeups could be the only thing that makes windows work... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131075/new/ https://reviews.llvm.org/D131075 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
