[Lldb-commits] [PATCH] D41871: [dotest] Remove crashinfo hook

2018-01-10 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL322167: [dotest] Remove crashinfo hook (authored by labath, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D41871?vs=129102&id=129224#toc Rep

[Lldb-commits] [PATCH] D41871: [dotest] Remove crashinfo hook

2018-01-09 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. Yup, this is a bit of complexity that it isn't worth supporting. https://reviews.llvm.org/D41871 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman

[Lldb-commits] [PATCH] D41871: [dotest] Remove crashinfo hook

2018-01-09 Thread Davide Italiano via Phabricator via lldb-commits
davide accepted this revision. davide added a comment. This revision is now accepted and ready to land. No objections from me. Thanks. https://reviews.llvm.org/D41871 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cg

[Lldb-commits] [PATCH] D41871: [dotest] Remove crashinfo hook

2018-01-09 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added reviewers: jingham, clayborg, davide. This used to be important when all tests were run in a single process, but that has no longer been the case for a while. Furthermore, this hook fails to build on new mac versions for several people, and it's not clear