Revision: 3518 Author: [email protected] Date: Tue Dec 22 05:10:24 2009 Log: Make the FastCloneShallowArrayStub a bit prettier.
[email protected] Review URL: http://codereview.chromium.org/507069 http://code.google.com/p/v8/source/detail?r=3518 Modified: /branches/bleeding_edge/src/ia32/codegen-ia32.cc ======================================= --- /branches/bleeding_edge/src/ia32/codegen-ia32.cc Tue Dec 22 04:41:45 2009 +++ /branches/bleeding_edge/src/ia32/codegen-ia32.cc Tue Dec 22 05:10:24 2009 @@ -6662,18 +6662,14 @@ // Load boilerplate object into ecx and check if we need to create a // boilerplate. + Label slow_case; __ mov(ecx, Operand(esp, 3 * kPointerSize)); __ mov(eax, Operand(esp, 2 * kPointerSize)); - ASSERT(kPointerSize == 4); - __ mov(ecx, FieldOperand(ecx, - eax, - times_2, - FixedArray::kHeaderSize)); + ASSERT((kPointerSize == 4) && (kSmiTagSize == 1) && (kSmiTag == 0)); + __ mov(ecx, FieldOperand(ecx, eax, times_2, FixedArray::kHeaderSize)); __ cmp(ecx, Factory::undefined_value()); - Label slow_case; __ j(equal, &slow_case); - // Allocate both the JS array and the elements array in one big // allocation. This avoids multiple limit checks. __ AllocateInNewSpace(size, eax, ebx, edx, &slow_case, TAG_OBJECT); -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
