[PATCH] D47913: [Support] Introduce a new utility for testing child process execution

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 7 17:08:21 PDT 2018


zturner added a reviewer: rnk.
zturner added a comment.

+rnk.

Also, one thing that's a little strange here is that this now marks the one and only Support test that is *not* a unit test.  Is that weird?  I feel like this is a much better pattern for testing process launching though, so I'd feel pretty bad if I had to re-write all this as unit tests.


https://reviews.llvm.org/D47913





More information about the llvm-commits mailing list