[Lldb-commits] [PATCH] D31823: Update LLDB Host to support IPv6 over TCP
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Sat Apr 15 08:06:56 PDT 2017
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
Thank you for taking the time to do this.
It looks like ppoll was added to linux quite a long time ago, but we still get some patches supporting surprisingly old kernels. I hope it doesn't come do that, but if it does, we can easily add a pselect-based fallback as well.
looks good, assuming that the SYS_EVENT_H define can go away.
Comment at: source/Host/common/MainLoop.cpp:22
+#define HAVE_SYS_EVENT_H 0
This looks wrong. Is this left over from testing ?
Comment at: source/Host/common/MainLoop.cpp:241
+ auto it = m_read_fds.find(event_list[i].ident);
+ if (it == m_read_fds.end())
Nit: the if could be extracted out of the `#ifdef`
More information about the lldb-commits