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
-~----------~----~----~----~------~----~------~--~---

Reply via email to