Looks great!  A couple of drive-by nits.

http://codereview.chromium.org/506037/diff/4001/5003
File bleeding_edge/src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/506037/diff/4001/5003#newcode3598
bleeding_edge/src/ia32/codegen-ia32.cc:3598: // Use the fast case
closure allocation code that allocated in new
allocated -> allocates

http://codereview.chromium.org/506037/diff/4001/5007
File bleeding_edge/src/objects.h (right):

http://codereview.chromium.org/506037/diff/4001/5007#newcode1807
bleeding_edge/src/objects.h:1807: return( descriptor_number << 1) + 1;
Not your code, but move parenthesis.

http://codereview.chromium.org/506037

-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to