russell.gallop accepted this revision.
russell.gallop added a comment.
This revision is now accepted and ready to land.

Would be nice if the test could check that the pid was being set to something 
but not sure how you could do that (beyond checking that it's a number).

Apart from that, LGTM.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78027/new/

https://reviews.llvm.org/D78027



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to