LGTM with comments.
Thanks, Vitaly http://codereview.chromium.org/6677076/diff/1/src/deoptimizer.cc File src/deoptimizer.cc (right): http://codereview.chromium.org/6677076/diff/1/src/deoptimizer.cc#newcode930 src/deoptimizer.cc:930: static Mutex* flag_mutex = OS::CreateMutex(); Can be removed. http://codereview.chromium.org/6677076/diff/1/test/cctest/test-macro-assembler-x64.cc File test/cctest/test-macro-assembler-x64.cc (right): http://codereview.chromium.org/6677076/diff/1/test/cctest/test-macro-assembler-x64.cc#newcode2100 test/cctest/test-macro-assembler-x64.cc:2100: <<<<<<< .working Huh? http://codereview.chromium.org/6677076/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
