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

Side by Side Diff: src/mips/codegen-mips.cc

Issue 1696043002: [runtime] Unify and simplify how frames are marked (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix merge problems Created 4 years, 9 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
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 #include "src/mips/codegen-mips.h" 5 #include "src/mips/codegen-mips.h"
6 6
7 #if V8_TARGET_ARCH_MIPS 7 #if V8_TARGET_ARCH_MIPS
8 8
9 #include "src/codegen.h" 9 #include "src/codegen.h"
10 #include "src/macro-assembler.h" 10 #include "src/macro-assembler.h"
(...skipping 1179 matching lines...) Expand 10 before | Expand all | Expand 10 after
1190 DCHECK(young_sequence_.length() == kNoCodeAgeSequenceLength); 1190 DCHECK(young_sequence_.length() == kNoCodeAgeSequenceLength);
1191 // Since patcher is a large object, allocate it dynamically when needed, 1191 // Since patcher is a large object, allocate it dynamically when needed,
1192 // to avoid overloading the stack in stress conditions. 1192 // to avoid overloading the stack in stress conditions.
1193 // DONT_FLUSH is used because the CodeAgingHelper is initialized early in 1193 // DONT_FLUSH is used because the CodeAgingHelper is initialized early in
1194 // the process, before MIPS simulator ICache is setup. 1194 // the process, before MIPS simulator ICache is setup.
1195 base::SmartPointer<CodePatcher> patcher( 1195 base::SmartPointer<CodePatcher> patcher(
1196 new CodePatcher(isolate, young_sequence_.start(), 1196 new CodePatcher(isolate, young_sequence_.start(),
1197 young_sequence_.length() / Assembler::kInstrSize, 1197 young_sequence_.length() / Assembler::kInstrSize,
1198 CodePatcher::DONT_FLUSH)); 1198 CodePatcher::DONT_FLUSH));
1199 PredictableCodeSizeScope scope(patcher->masm(), young_sequence_.length()); 1199 PredictableCodeSizeScope scope(patcher->masm(), young_sequence_.length());
1200 patcher->masm()->Push(ra, fp, cp, a1); 1200 patcher->masm()->PushStandardFrame(a1);
1201 patcher->masm()->nop(Assembler::CODE_AGE_SEQUENCE_NOP); 1201 patcher->masm()->nop(Assembler::CODE_AGE_SEQUENCE_NOP);
1202 patcher->masm()->Addu(
1203 fp, sp, Operand(StandardFrameConstants::kFixedFrameSizeFromFp));
1204 } 1202 }
1205 1203
1206 1204
1207 #ifdef DEBUG 1205 #ifdef DEBUG
1208 bool CodeAgingHelper::IsOld(byte* candidate) const { 1206 bool CodeAgingHelper::IsOld(byte* candidate) const {
1209 return Memory::uint32_at(candidate) == kCodeAgePatchFirstInstruction; 1207 return Memory::uint32_at(candidate) == kCodeAgePatchFirstInstruction;
1210 } 1208 }
1211 #endif 1209 #endif
1212 1210
1213 1211
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
1259 } 1257 }
1260 } 1258 }
1261 1259
1262 1260
1263 #undef __ 1261 #undef __
1264 1262
1265 } // namespace internal 1263 } // namespace internal
1266 } // namespace v8 1264 } // namespace v8
1267 1265
1268 #endif // V8_TARGET_ARCH_MIPS 1266 #endif // V8_TARGET_ARCH_MIPS
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698