tberghammer accepted this revision. tberghammer added a comment. This revision is now accepted and ready to land.
Looks good with 2 minor comments - After the renames the test path don't say that the test is thread related. I think you should add it back to somewhere (e.g. to the test name) - Should we test the thread destroy during instruction single stepping scenario also? Can it hit the same or a similar issue? http://reviews.llvm.org/D12280 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits