[PATCH] D49501: Remove scheduling dependency from XRay :: Posix/fork_basic_logging.cc

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 18 16:48:53 PDT 2018


dberris added a comment.

In https://reviews.llvm.org/D49501#1167385, @ormris wrote:

> Thanks for the review! Before I commit this, would it be possible to change the second part of the test (after line 82) to be scheduling independent?


The secondary test already should do that, since it's already looking for independent process markers. If it's not doing the right thing, then we (@Maknee or I) should look into how to re-structure this test to be less dependent on scheduling (maybe looking for a different function called from the child and the parent).


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49501





More information about the llvm-commits mailing list