http://codereview.chromium.org/164472/diff/1/2 File src/x64/assembler-x64.cc (right):
http://codereview.chromium.org/164472/diff/1/2#newcode1611 Line 1611: EnsureSpace ensure_space(this); On 2009/08/13 12:12:27, Lasse Reichstein wrote: > Move ensure_space below the call to testb, which has its own ensure_space. Done. http://codereview.chromium.org/164472/diff/1/2#newcode1613 Line 1613: if ((mask.value_ & 0xFFFFFF00) == 0) { On 2009/08/13 12:12:27, Lasse Reichstein wrote: > Use is_uint8 to test. Done. http://codereview.chromium.org/164472/diff/1/2#newcode1631 Line 1631: EnsureSpace ensure_space(this); On 2009/08/13 12:12:27, Lasse Reichstein wrote: > Same comments as above. Done. http://codereview.chromium.org/164472 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
