[PATCH] D24111: [lit] Fail testing if a googletest executable crashes during test discovery

[Daniel Dunbar via llvm-commits]

ddunbar added a comment.

LGTM.

There might be clients of the capture() API which might aren't expecting it to throw on error, and will need to be updated, but I agree this makes more sense.


https://reviews.llvm.org/D24111