Re: [Lldb-commits] [lldb] r257946 - Fix TestDebugBreak.py.

2016-01-15 Thread Siva Chandra via lldb-commits
Thanks a lot. On Fri, Jan 15, 2016 at 3:26 PM, Zachary Turner wrote: > Sure > > On Fri, Jan 15, 2016 at 3:25 PM Siva Chandra wrote: >> >> SGTM. You want to do it? >> >> On Fri, Jan 15, 2016 at 3:24 PM, Zachary Turner >> wrote: >> > How about `lldb.eStopReasonException if osIsWindows() else >> >

Re: [Lldb-commits] [lldb] r257946 - Fix TestDebugBreak.py.

2016-01-15 Thread Zachary Turner via lldb-commits
Sure On Fri, Jan 15, 2016 at 3:25 PM Siva Chandra wrote: > SGTM. You want to do it? > > On Fri, Jan 15, 2016 at 3:24 PM, Zachary Turner > wrote: > > How about `lldb.eStopReasonException if osIsWindows() else > > lldb.eStopReasonSignal`? > > > > Seems like that should work for everyone? > > > >

Re: [Lldb-commits] [lldb] r257946 - Fix TestDebugBreak.py.

2016-01-15 Thread Siva Chandra via lldb-commits
SGTM. You want to do it? On Fri, Jan 15, 2016 at 3:24 PM, Zachary Turner wrote: > How about `lldb.eStopReasonException if osIsWindows() else > lldb.eStopReasonSignal`? > > Seems like that should work for everyone? > > On Fri, Jan 15, 2016 at 3:20 PM Siva Chandra wrote: >> >> This fails for i386

Re: [Lldb-commits] [lldb] r257946 - Fix TestDebugBreak.py.

2016-01-15 Thread Zachary Turner via lldb-commits
How about `lldb.eStopReasonException if osIsWindows() else lldb.eStopReasonSignal`? Seems like that should work for everyone? On Fri, Jan 15, 2016 at 3:20 PM Siva Chandra wrote: > This fails for i386 (not enabled for x86_64 anyway): > > http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04

Re: [Lldb-commits] [lldb] r257946 - Fix TestDebugBreak.py.

2016-01-15 Thread Siva Chandra via lldb-commits
This fails for i386 (not enabled for x86_64 anyway): http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/10400 If I replace lldb.eStopReasonException with lldb.eStopReasonSignal on my local machine, it works. On Fri, Jan 15, 2016 at 2:22 PM, Zachary Turner via lldb-commits wr