Lgtm!
http://codereview.chromium.org/18363/diff/1/6 File src/bytecodes-irregexp.h (right): http://codereview.chromium.org/18363/diff/1/6#newcode40 Line 40: V(BREAK, 0, 4) /* bc8 */ \ Why not count in whole words? http://codereview.chromium.org/18363/diff/1/3 File src/regexp-macro-assembler-irregexp-inl.h (right): http://codereview.chromium.org/18363/diff/1/3#newcode39 Line 39: void RegExpMacroAssemblerIrregexp::Emit(uint32_t byte, uint32_t twenty_four_bits) { Maybe assert that twenty_four_bits does indeed only use 24 bits? http://codereview.chromium.org/18363 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
