LGTM if you re-staticize the spare bugger.
http://codereview.chromium.org/113631/diff/1/2 File src/ia32/assembler-ia32.cc (right): http://codereview.chromium.org/113631/diff/1/2#newcode291 Line 291: byte* Assembler::spare_buffer_ = NULL; I think this one should be static. It's for reusing a buffer between different assembler instances, to avoid allocating a new one every time. http://codereview.chromium.org/113631 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
