https://chromiumcodereview.appspot.com/12047044/diff/16001/src/mark-compact.h
File src/mark-compact.h (right):

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/mark-compact.h#newcode693
src/mark-compact.h:693: void MarkInParallel();
On 2013/01/31 14:00:04, Michael Starzinger wrote:
This is getting clunky, can we group these functions into a "//
Parallel
sweeping support." and a "// Parallel marking support." section?

Done.

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/marking-thread.cc
File src/marking-thread.cc (right):

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/marking-thread.cc#newcode1
src/marking-thread.cc:1: // Copyright 2012 the V8 project authors. All
rights reserved.
On 2013/01/31 14:00:04, Michael Starzinger wrote:
2013

Done.

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/marking-thread.h
File src/marking-thread.h (right):

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/marking-thread.h#newcode1
src/marking-thread.h:1: // Copyright 2012 the V8 project authors. All
rights reserved.
On 2013/01/31 14:00:04, Michael Starzinger wrote:
2013

Done.

https://chromiumcodereview.appspot.com/12047044/diff/16001/src/marking-thread.h#newcode38
src/marking-thread.h:38: #include "heap.h"
On 2013/01/31 14:00:04, Michael Starzinger wrote:
Is it possible to alpha-sort the includes or does that cause
dependency errors?

It results in dependency errors.

https://chromiumcodereview.appspot.com/12047044/

--
--
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/groups/opt_out.


Reply via email to