[Lldb-commits] [PATCH] D32753: MainLoop: Add unit tests

2017-05-04 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL302133: MainLoop: Add unit tests (authored by labath). Changed prior to commit: https://reviews.llvm.org/D32753?vs=97464&id=97802#toc Repository: rL LLVM https://reviews.llvm.org/D32753 Files: lld

[Lldb-commits] [PATCH] D32753: MainLoop: Add unit tests

2017-05-03 Thread Eugene Zemtsov via Phabricator via lldb-commits
eugene accepted this revision. eugene added inline comments. This revision is now accepted and ready to land. Comment at: source/Host/common/MainLoop.cpp:66 class MainLoop::RunImpl { public: Could you please add here a comment describing when kqueue/pselect/

[Lldb-commits] [PATCH] D32753: MainLoop: Add unit tests

2017-05-02 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. Herald added a subscriber: mgorny. This adds a couple of unit tests to the MainLoop class. To get the kqueue based version of the signal handling passing, I needed to modify the implementation a bit to make the queue object persistent. Otherwise, only the signals whic