LGTM.
http://codereview.chromium.org/39014/diff/1/8 File src/code-stubs.cc (right): http://codereview.chromium.org/39014/diff/1/8#newcode74 Line 74: Print(); Do you want to both Print and Disassemble? http://codereview.chromium.org/39014/diff/1/5 File src/stub-cache.cc (right): http://codereview.chromium.org/39014/diff/1/5#newcode914 Line 914: return GetCodeWithFlags(Code::ComputeMonomorphicFlags(Code::LOAD_IC, type), Maybe this would look better if you precomputed the flags and stored that in a local variable? Goes for a lot of code in here. http://codereview.chromium.org/39014/diff/1/5#newcode941 Line 941: Code::Flags flags = Code::ComputeMonomorphicFlags(Code::CALL_IC, type, argc); Yes. Like this. http://codereview.chromium.org/39014 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
