Unless you make significant other changes, feel free to just commit.  I
don't have any other concerns.  Thanks for working on this!

On Tue, Sep 29, 2015 at 12:51 PM Todd Fiala <todd.fi...@gmail.com> wrote:

> tfiala added a comment.
>
> > Does that look right to you?
>
>
> That result looks right - the two soft terminate tests get skipped when
> it's not reported.
>
> If you ran the lldb test suite with that setup, they should work as well
> as they did before the change.  (If they don't, I'm missing a lower level
> test).
>
> I'll fix up those other parts you called out and get what is hopefully the
> final patch up.
>
>
> http://reviews.llvm.org/D13124
>
>
>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to