https://codereview.chromium.org/1230303005/diff/120001/src/execution.cc
File src/execution.cc (right):
https://codereview.chromium.org/1230303005/diff/120001/src/execution.cc#newcode418
src/execution.cc:418: if (isolate_->futex_wait_list_node()->waiting()) {
On 2015/08/19 at 05:26:05, Jarin wrote:
Could we perhaps pull the waiting() check into NotifyWake?
Then the waiting accessors could be private (and perhaps we could even
pull the waiting_ flag into the critical section and make it
non-atomic).
My thought here was to do a quick check to avoid taking a lock if there
is no futex waiting (which is the common case). What do you think?
https://codereview.chromium.org/1230303005/diff/120001/src/futex-emulation.h
File src/futex-emulation.h (right):
https://codereview.chromium.org/1230303005/diff/120001/src/futex-emulation.h#newcode44
src/futex-emulation.h:44: waiting_(false),
On 2015/08/19 at 05:26:05, Jarin wrote:
waiting_(0)
Done.
https://codereview.chromium.org/1230303005/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
