labath added a comment.

This looks like a tricky bug. Thanks for tracking it down.

I have two questions though: :D

- could you use c++11 locking primitives in the test? pthreads do not work on 
windows
- what exactly will be the effect of this? Will we start all threads 
immediately, or will we do the usual dance of trying to run one thread for a 
short period of time, and then resume all of them if that times out? If it's 
the latter, I'm wondering if we really need this call detection logic -- we 
could theoretically just use the "run all" method all the time (except if 
explicitly disabled), with the knowledge that all reasonable call-less 
sequences will finish before the "run single thread" timeout expires. And it 
would automatically also handle weird call-less sequences with spinlocks or 
what not...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71440



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

Reply via email to