LGTM apart from the comments.
http://codereview.chromium.org/147022/diff/1/2 File src/heap.cc (right): http://codereview.chromium.org/147022/diff/1/2#newcode1291 Line 1291: // gcc-4.4 has problem to generate the correct vtables if the following Should be "gcc-4.4 has problems generating..." I don't understand the comment. It doesn't seem like vtables are being generated here. http://codereview.chromium.org/147022/diff/1/2#newcode1300 Line 1300: // Possible gcc bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32327 Is this relevant? According to the bug this was fixed in gcc-4.3. http://codereview.chromium.org/147022 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---