http://codereview.chromium.org/597021/diff/1/3 File src/fast-codegen.h (right):
http://codereview.chromium.org/597021/diff/1/3#newcode102 src/fast-codegen.h:102: bool is_smi(Register reg) { return (smi_bits_ & reg.bit()) != 0; } What happens if reg == no_reg? Maybe we should ASSERT(reg != noreg) here, since reg.bit() of no_reg is -1. http://codereview.chromium.org/597021/diff/1/3#newcode138 src/fast-codegen.h:138: uint32_t smi_bits_; smi_bits_ should be initialized to 0? http://codereview.chromium.org/597021/diff/1/4 File src/ia32/fast-codegen-ia32.cc (right): http://codereview.chromium.org/597021/diff/1/4#newcode115 src/ia32/fast-codegen-ia32.cc:115: if (is_smi(accumulator0())) { This condition should be always true, since there is no write barrier generated, right? http://codereview.chromium.org/597021 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
