One follow-up comment about the test case.
https://codereview.chromium.org/686273005/diff/20001/test/cctest/compiler/test-scheduler.cc
File test/cctest/compiler/test-scheduler.cc (right):
https://codereview.chromium.org/686273005/diff/20001/test/cctest/compiler/test-scheduler.cc#newcode169
test/cctest/compiler/test-scheduler.cc:169: block->set_deferred(i & 1);
As discussed offline: This test isn't particularly useful, because these
blocks form a chain and almost all of them will end up being deferred.
Maybe we should revert this line and add one specific test case that
tests the propagation of the "deferredness" explicitly.
https://codereview.chromium.org/686273005/
--
--
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.
