[Lldb-commits] [PATCH] Windows Build Fixes

2013-10-18 Thread Deepak Panickal
Some of the new commits have broken the Windows build. Fixed the build. http://llvm-reviews.chandlerc.com/D1976 Files: source/DataFormatters/FormatManager.cpp source/Host/common/SocketAddress.cpp source/Plugins/DynamicLoader/POSIX-DYLD/AuxVector.cpp

Re: [Lldb-commits] [PATCH] Windows Build Fixes

2013-10-18 Thread Deepak Panickal
Removed __func__ http://llvm-reviews.chandlerc.com/D1976 CHANGE SINCE LAST DIFF http://llvm-reviews.chandlerc.com/D1976?vs=5029id=5030#toc Files: source/DataFormatters/FormatManager.cpp source/Host/common/SocketAddress.cpp source/Plugins/DynamicLoader/POSIX-DYLD/AuxVector.cpp

[Lldb-commits] [lldb] r192987 - Make sure the CallFunction Thread plans don't try to do DoTakedown if their thread

2013-10-18 Thread Jim Ingham
Author: jingham Date: Fri Oct 18 12:11:02 2013 New Revision: 192987 URL: http://llvm.org/viewvc/llvm-project?rev=192987view=rev Log: Make sure the CallFunction Thread plans don't try to do DoTakedown if their thread has gone away by the time they get around to doing it. rdar://problem/15245544

[Lldb-commits] [lldb] r192993 - rdar://problem/15182550

2013-10-18 Thread Enrico Granata
Author: enrico Date: Fri Oct 18 13:57:49 2013 New Revision: 192993 URL: http://llvm.org/viewvc/llvm-project?rev=192993view=rev Log: rdar://problem/15182550 Removing Host/Atomic.h This header file was not being copied as part of our public API headers and this in turn was causing any plugin to

[Lldb-commits] [lldb] r192994 - Add stub FreeBSD ProcessMonitor::ReadThreadPointer

2013-10-18 Thread Ed Maste
Author: emaste Date: Fri Oct 18 14:16:44 2013 New Revision: 192994 URL: http://llvm.org/viewvc/llvm-project?rev=192994view=rev Log: Add stub FreeBSD ProcessMonitor::ReadThreadPointer Modified: lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp

Re: [Lldb-commits] [PATCH] Windows Build Fixes

2013-10-18 Thread João Matos
LGTM with changes suggested by Greg and Ed. By the way, to make future reviews easier you can add some more context in the patch, see: http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface http://llvm-reviews.chandlerc.com/D1976