[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())
+        continue;
Nit: the if could be extracted out of the `#ifdef`


More information about the lldb-commits mailing list