[PATCH] Workaround ASAN race condition in future tests by manually joining threads.

Jonathan Roelofs jonathan at codesourcery.com
Thu Nov 20 16:14:59 PST 2014


So 4 things:
  *  I'm a little out of my comfort zone with this thread stuff... pretty far from my area of expertise
  *  Perhaps you missed a few, for example: thread/futures/futures.promise/set_value_at_thread_exit_void.pass.cpp (grep for 'detach' in tests).
  *  Doesn't this reduce the test coverage for the thread.detach() method?
  *  I *really* *really* don't like the idea of changing tests to push bugs under the rug. It'd be much better to 'XFAIL: asan' these. I have a thought on how to fix asan that I'll add to the PR.


Cheers,

Jon

http://reviews.llvm.org/D6347






More information about the cfe-commits mailing list