jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land.
I marked a comment left over from a previous draft of the patch that isn't needed. Other than that, this looks fine. ================ Comment at: source/Target/Process.cpp:3561-3562 @@ +3560,4 @@ + Error error; + // Cannot resume already exited process. Public running lock is + // held when PrivateResume is entered + if (!IsAlive()) ---------------- Is this comment needed anymore? http://reviews.llvm.org/D17635 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits