Landing.
http://codereview.chromium.org/7324049/diff/1/src/incremental-marking.cc File src/incremental-marking.cc (right): http://codereview.chromium.org/7324049/diff/1/src/incremental-marking.cc#newcode541 src/incremental-marking.cc:541: if (!FLAG_incremental_marking_steps) return; On 2011/07/12 08:18:32, Lasse Reichstein wrote:
Are FLAG_incremental_marking and FLAG_incremental_marking_steps forced
to be
dependent in any way? Should they be (i.e., it seems it should be
sufficient to
check FLAG_incremental_marking_steps)?
They are independent. http://codereview.chromium.org/7324049/diff/1/src/incremental-marking.cc#newcode542 src/incremental-marking.cc:542: if (state_ != SWEEPING && state_ != MARKING) return; On 2011/07/12 08:18:32, Lasse Reichstein wrote:
How about combining all these conditions with || and just having one
return? Done. http://codereview.chromium.org/7324049/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
