LGTM with comments
http://codereview.chromium.org/11189091/diff/1/src/deoptimizer.cc File src/deoptimizer.cc (right): http://codereview.chromium.org/11189091/diff/1/src/deoptimizer.cc#newcode1454 src/deoptimizer.cc:1454: while (!current->IsUndefined()) { Why not current != undefined, if you already have it? http://codereview.chromium.org/11189091/diff/1/src/deoptimizer.cc#newcode1469 src/deoptimizer.cc:1469: if (head != context->get(Context::OPTIMIZED_FUNCTIONS_LIST)) { If this extra compare just to avoid a write barrier? Otherwise, why not simply always set the head? http://codereview.chromium.org/11189091/diff/1/src/deoptimizer.cc#newcode1485 src/deoptimizer.cc:1485: while (!current->IsUndefined()) { Why not current != undefined, if you already have it? http://codereview.chromium.org/11189091/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
