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

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 12 16:14:02 PDT 2018


dblaikie accepted this revision.
dblaikie added a comment.

ah, yeah - thanks for pointing out/explaining the existing tests a bit & how they're re-executing the full gtest then waiting to be run themselves, etc.

Worth having /some/ binary rather than rerunning through gtest, I think - and once you have one, this doesn't seem too bad.

Could you port/move the existing unit tests over to this framework so there's not a mix of approaches to the same problem?


https://reviews.llvm.org/D47913





More information about the llvm-commits mailing list