LGTM. Seems easy to extend and understand.
http://codereview.chromium.org/8835/diff/1/12 File src/codegen-ia32.cc (right): http://codereview.chromium.org/8835/diff/1/12#newcode817 Line 817: flags = ((loop_nesting_ > 0) && type->IsLikelySmi()) ? SMI_CODE_INLINED : Nit: I would move ? onto a new line and : onto yet another new line, like x = <complicated boolean expression> ? <blah> : <argh>; http://codereview.chromium.org/8835/diff/1/5 File src/variables.h (right): http://codereview.chromium.org/8835/diff/1/5#newcode83 Line 83: static char* Type2String(StaticType* type) { I would move this to a .cc file. http://codereview.chromium.org/8835/diff/1/5#newcode95 Line 95: // LIKELY_SMI accessors How about MarkAsLikelySmi? Set sounds like something that takes an argument... http://codereview.chromium.org/8835 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
