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; On 2009/05/20 12:00:25, Lasse Reichstein wrote: > 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. The declaration in the class definition is static. The definition does not need the keyword static, and static means something different if it is used here (file-scoped visibility). http://codereview.chromium.org/113631 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
