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

Side by Side Diff: src/mips/code-stubs-mips.cc

Issue 2409553003: [heap] Remove dead write write barrier counter code (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « src/heap/spaces.cc ('k') | src/mips64/code-stubs-mips64.cc » ('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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #if V8_TARGET_ARCH_MIPS 5 #if V8_TARGET_ARCH_MIPS
6 6
7 #include "src/code-stubs.h" 7 #include "src/code-stubs.h"
8 #include "src/api-arguments.h" 8 #include "src/api-arguments.h"
9 #include "src/base/bits.h" 9 #include "src/base/bits.h"
10 #include "src/bootstrapper.h" 10 #include "src/bootstrapper.h"
(...skipping 3165 matching lines...) Expand 10 before | Expand all | Expand 10 after
3176 3176
3177 3177
3178 void RecordWriteStub::CheckNeedsToInformIncrementalMarker( 3178 void RecordWriteStub::CheckNeedsToInformIncrementalMarker(
3179 MacroAssembler* masm, 3179 MacroAssembler* masm,
3180 OnNoNeedToInformIncrementalMarker on_no_need, 3180 OnNoNeedToInformIncrementalMarker on_no_need,
3181 Mode mode) { 3181 Mode mode) {
3182 Label on_black; 3182 Label on_black;
3183 Label need_incremental; 3183 Label need_incremental;
3184 Label need_incremental_pop_scratch; 3184 Label need_incremental_pop_scratch;
3185 3185
3186 __ And(regs_.scratch0(), regs_.object(), Operand(~Page::kPageAlignmentMask));
3187 __ lw(regs_.scratch1(),
3188 MemOperand(regs_.scratch0(),
3189 MemoryChunk::kWriteBarrierCounterOffset));
3190 __ Subu(regs_.scratch1(), regs_.scratch1(), Operand(1));
3191 __ sw(regs_.scratch1(),
3192 MemOperand(regs_.scratch0(),
3193 MemoryChunk::kWriteBarrierCounterOffset));
3194 __ Branch(&need_incremental, lt, regs_.scratch1(), Operand(zero_reg));
3195
3196 // Let's look at the color of the object: If it is not black we don't have 3186 // Let's look at the color of the object: If it is not black we don't have
3197 // to inform the incremental marker. 3187 // to inform the incremental marker.
3198 __ JumpIfBlack(regs_.object(), regs_.scratch0(), regs_.scratch1(), &on_black); 3188 __ JumpIfBlack(regs_.object(), regs_.scratch0(), regs_.scratch1(), &on_black);
3199 3189
3200 regs_.Restore(masm); 3190 regs_.Restore(masm);
3201 if (on_no_need == kUpdateRememberedSetOnNoNeedToInformIncrementalMarker) { 3191 if (on_no_need == kUpdateRememberedSetOnNoNeedToInformIncrementalMarker) {
3202 __ RememberedSetHelper(object(), 3192 __ RememberedSetHelper(object(),
3203 address(), 3193 address(),
3204 value(), 3194 value(),
3205 save_fp_regs_mode(), 3195 save_fp_regs_mode(),
(...skipping 1904 matching lines...) Expand 10 before | Expand all | Expand 10 after
5110 kStackUnwindSpace, kInvalidStackOffset, 5100 kStackUnwindSpace, kInvalidStackOffset,
5111 return_value_operand, NULL); 5101 return_value_operand, NULL);
5112 } 5102 }
5113 5103
5114 #undef __ 5104 #undef __
5115 5105
5116 } // namespace internal 5106 } // namespace internal
5117 } // namespace v8 5107 } // namespace v8
5118 5108
5119 #endif // V8_TARGET_ARCH_MIPS 5109 #endif // V8_TARGET_ARCH_MIPS
OLDNEW
« no previous file with comments | « src/heap/spaces.cc ('k') | src/mips64/code-stubs-mips64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698