[PATCH] D37563: Convenience/safety fix for llvm::sys::Execute(And|No)Wait

[Alexander Kornienko via Phabricator via llvm-commits]

alexfh added a comment.

In https://reviews.llvm.org/D37563#865541, @vsk wrote:

> It'd be nice to add in "expected 3 redirects" to the assert bodies.


Sorry, missed this comment somehow. If we add a message, it should also cover the case of zero redirects. Something along the lines of "expected 0 or 3 redirects". I have a slight concern that the message would basically repeat the condition resulting in a sort of tautology (`assert((Redirects.empty() || Redirects.size() == 3) && "expected 0 or 3 redirects");`). But if you still find this useful, I can add a message.


Repository:
  rL LLVM

https://reviews.llvm.org/D37563