Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(662)

Side by Side Diff: src/ia32/stub-cache-ia32.cc

Issue 7247002: Estimate a (close) upper bound on the size of black-marked objects on each page. (Closed) Base URL: https://v8.googlecode.com/svn/branches/experimental/gc
Patch Set: Address review comments. Make compile on x64. Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/ia32/macro-assembler-ia32.cc ('k') | src/incremental-marking.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 1527 matching lines...) Expand 10 before | Expand all | Expand 10 after
1538 // don't need to worry about the holes because they are in old space and 1538 // don't need to worry about the holes because they are in old space and
1539 // already marked black. 1539 // already marked black.
1540 __ RecordWrite(ebx, edx, ecx, kDontSaveFPRegs, OMIT_REMEMBERED_SET); 1540 __ RecordWrite(ebx, edx, ecx, kDontSaveFPRegs, OMIT_REMEMBERED_SET);
1541 1541
1542 // Restore receiver to edx as finish sequence assumes it's here. 1542 // Restore receiver to edx as finish sequence assumes it's here.
1543 __ mov(edx, Operand(esp, (argc + 1) * kPointerSize)); 1543 __ mov(edx, Operand(esp, (argc + 1) * kPointerSize));
1544 1544
1545 // Increment element's and array's sizes. 1545 // Increment element's and array's sizes.
1546 __ add(FieldOperand(ebx, FixedArray::kLengthOffset), 1546 __ add(FieldOperand(ebx, FixedArray::kLengthOffset),
1547 Immediate(Smi::FromInt(kAllocationDelta))); 1547 Immediate(Smi::FromInt(kAllocationDelta)));
1548
1549 // NOTE: This only happen in new-space, where we don't
1550 // care about the black-byte-count on pages. Otherwise we should
1551 // update that too if the object is black.
1552
1548 __ mov(FieldOperand(edx, JSArray::kLengthOffset), eax); 1553 __ mov(FieldOperand(edx, JSArray::kLengthOffset), eax);
1549 1554
1550 __ ret((argc + 1) * kPointerSize); 1555 __ ret((argc + 1) * kPointerSize);
1551 } 1556 }
1552 1557
1553 __ bind(&call_builtin); 1558 __ bind(&call_builtin);
1554 __ TailCallExternalReference( 1559 __ TailCallExternalReference(
1555 ExternalReference(Builtins::c_ArrayPush, isolate()), 1560 ExternalReference(Builtins::c_ArrayPush, isolate()),
1556 argc + 1, 1561 argc + 1,
1557 1); 1562 1);
(...skipping 2310 matching lines...) Expand 10 before | Expand all | Expand 10 after
3868 masm->isolate()->builtins()->KeyedStoreIC_MissForceGeneric(); 3873 masm->isolate()->builtins()->KeyedStoreIC_MissForceGeneric();
3869 __ jmp(ic_force_generic, RelocInfo::CODE_TARGET); 3874 __ jmp(ic_force_generic, RelocInfo::CODE_TARGET);
3870 } 3875 }
3871 3876
3872 3877
3873 #undef __ 3878 #undef __
3874 3879
3875 } } // namespace v8::internal 3880 } } // namespace v8::internal
3876 3881
3877 #endif // V8_TARGET_ARCH_IA32 3882 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « src/ia32/macro-assembler-ia32.cc ('k') | src/incremental-marking.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698