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

Unified Diff: src/ppc/lithium-codegen-ppc.cc

Issue 1376333003: Eliminate no_frame_range data (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/mips64/lithium-codegen-mips64.cc ('k') | src/profiler/cpu-profiler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/ppc/lithium-codegen-ppc.cc
diff --git a/src/ppc/lithium-codegen-ppc.cc b/src/ppc/lithium-codegen-ppc.cc
index ad6d8db13d33dd0d4d3d9509ed2ec75708b674da..3d3b473de3525eeef84a3967dfe4adf2cd93b30e 100644
--- a/src/ppc/lithium-codegen-ppc.cc
+++ b/src/ppc/lithium-codegen-ppc.cc
@@ -152,7 +152,6 @@ bool LCodeGen::GeneratePrologue() {
__ Prologue(info()->IsCodePreAgingActive(), prologue_offset);
}
frame_is_built_ = true;
- info_->AddNoFrameRange(0, masm_->pc_offset());
}
// Reserve space for the stack slots needed by the code.
@@ -2845,12 +2844,11 @@ void LCodeGen::DoReturn(LReturn* instr) {
if (info()->saves_caller_doubles()) {
RestoreCallerDoubles();
}
- int no_frame_start = -1;
if (instr->has_constant_parameter_count()) {
int parameter_count = ToInteger32(instr->constant_parameter_count());
int32_t sp_delta = (parameter_count + 1) * kPointerSize;
if (NeedsEagerFrame()) {
- no_frame_start = masm_->LeaveFrame(StackFrame::JAVA_SCRIPT, sp_delta);
+ masm_->LeaveFrame(StackFrame::JAVA_SCRIPT, sp_delta);
} else if (sp_delta != 0) {
__ addi(sp, sp, Operand(sp_delta));
}
@@ -2859,17 +2857,13 @@ void LCodeGen::DoReturn(LReturn* instr) {
Register reg = ToRegister(instr->parameter_count());
// The argument count parameter is a smi
if (NeedsEagerFrame()) {
- no_frame_start = masm_->LeaveFrame(StackFrame::JAVA_SCRIPT);
+ masm_->LeaveFrame(StackFrame::JAVA_SCRIPT);
}
__ SmiToPtrArrayOffset(r0, reg);
__ add(sp, sp, r0);
}
__ blr();
-
- if (no_frame_start != -1) {
- info_->AddNoFrameRange(no_frame_start, masm_->pc_offset());
- }
}
« no previous file with comments | « src/mips64/lithium-codegen-mips64.cc ('k') | src/profiler/cpu-profiler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698