http://codereview.chromium.org/27269/diff/1/2 File src/heap.cc (right):
http://codereview.chromium.org/27269/diff/1/2#newcode1634 Line 1634: ASSERT((obj_size % Code::kCodeAlignment) == 0); On 2009/02/27 10:32:11, Kasper Lund wrote: > IsAligned(obj_size, Code::kCodeAlignment) would be easier to read. Done. http://codereview.chromium.org/27269/diff/1/2#newcode1669 Line 1669: return code; On 2009/02/27 10:32:11, Kasper Lund wrote: > It would be nice if you could assert that the entry point (instruction start) is > aligned the way you want it. Done now as part of CodeVerify(). http://codereview.chromium.org/27269/diff/1/3 File src/objects.h (right): http://codereview.chromium.org/27269/diff/1/3#newcode2282 Line 2282: static const int kFiller6Offset = kKindSpecificFlagsOffset + kIntSize; On 2009/02/27 10:32:11, Kasper Lund wrote: > Comment on why you need fillers here? Done. http://codereview.chromium.org/27269 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
