amccarth added a comment.

In https://reviews.llvm.org/D26393#589421, @labath wrote:

> In https://reviews.llvm.org/D26393#589363, @amccarth wrote:
>
> > I started testing the new plugin on Windows yesterday, and it doesn't work 
> > (all the tests fail).  I'm planning to debug today, and, once I get it 
> > working, I'd be happy to switch it over.
>
>
> Interesting. It seemed to work fine for me. How did you test that?


I made changes very similar to this patch and ran check-lldb.  The tests for 
the new minidump plugin passed, but the original Windows ones all failed, 
apparently because it cannot find any threads in the dumps.  I'll have more 
time to look at it this afternoon.


https://reviews.llvm.org/D26393



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to