[Lldb-commits] [PATCH] D13636: Symbols::LocateExecutableSymbolFile() shouldn't try to look for files in /usr/lib/debug on Windows

2015-10-11 Thread Vadim Macagon via lldb-commits
enlight created this revision. enlight added reviewers: zturner, clayborg. enlight added a subscriber: lldb-commits. enlight set the repository for this revision to rL LLVM. `/usr/lib/debug` doesn't exist on Windows so there's no point even attempting to look for symbol files in there.

Re: [Lldb-commits] [PATCH] D13636: Symbols::LocateExecutableSymbolFile() shouldn't try to look for files in /usr/lib/debug on Windows

2015-10-11 Thread Vadim Macagon via lldb-commits
enlight added a comment. Zach, I've looked at the path separator issue you've mentioned earlier in D13201 , and I think it's fine as is. All those paths with the forward slashes end up in a FileSpec, which converts forward slashes to

Re: [Lldb-commits] [PATCH] D13448: Update swig generation shell scripts to run under Python 3.x

2015-10-11 Thread Zachary Turner via lldb-commits
Yes this went in On Sat, Oct 10, 2015 at 9:20 PM Todd Fiala wrote: > tfiala added a comment. > > Hey Zachary, I think this one went in, didn't it? Can you close it out if > so? Thanks! > > > http://reviews.llvm.org/D13448 > > > >

Re: [Lldb-commits] [PATCH] D13448: Update swig generation shell scripts to run under Python 3.x

2015-10-11 Thread Todd Fiala via lldb-commits
tfiala closed this revision. tfiala added a comment. Great! Closing out. http://reviews.llvm.org/D13448 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D13638: Add a missing Mutex Locker in ResetCurrentInlinedDepth

2015-10-11 Thread Keno Fischer via lldb-commits
loladiro created this revision. loladiro added reviewers: clayborg, jingham. loladiro added a subscriber: lldb-commits. loladiro set the repository for this revision to rL LLVM. This function reads m_frames, so it needs to lock the mutex like all other function that access m_frames. Found using

Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-10-11 Thread Zachary Turner via lldb-commits
Yes On Sat, Oct 10, 2015 at 9:18 PM Todd Fiala wrote: > tfiala added a comment. > > Did this go in, Zachary? I'm just looking to clear out my reviews and > this one seems to still be open. > > > http://reviews.llvm.org/D12612 > > > >

Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-10-11 Thread Todd Fiala via lldb-commits
tfiala closed this revision. tfiala added a comment. Great, thanks! http://reviews.llvm.org/D12612 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D13334: Preliminary NetBSD support

2015-10-11 Thread Kamil Rytarowski via lldb-commits
krytarowski updated this revision to Diff 37077. Repository: rL LLVM http://reviews.llvm.org/D13334 Files: include/lldb/Host/Config.h include/lldb/Host/HostInfo.h include/lldb/Host/HostNativeThread.h include/lldb/Host/HostNativeThreadForward.h include/lldb/Host/netbsd/Config.h