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

Side by Side Diff: src/deoptimizer.cc

Issue 2423323002: [builtins] Remove the unused AllocationSite slot from ConstructFrame. (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/builtins/x64/builtins-x64.cc ('k') | src/frames.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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/deoptimizer.h" 5 #include "src/deoptimizer.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "src/accessors.h" 9 #include "src/accessors.h"
10 #include "src/ast/prettyprinter.h" 10 #include "src/ast/prettyprinter.h"
(...skipping 1584 matching lines...) Expand 10 before | Expand all | Expand 10 after
1595 output_frame->SetFrameSlot(output_offset, value); 1595 output_frame->SetFrameSlot(output_offset, value);
1596 DebugPrintOutputSlot(value, frame_index, output_offset, 1596 DebugPrintOutputSlot(value, frame_index, output_offset,
1597 "typed frame marker\n"); 1597 "typed frame marker\n");
1598 1598
1599 // The context can be gotten from the previous frame. 1599 // The context can be gotten from the previous frame.
1600 output_offset -= kPointerSize; 1600 output_offset -= kPointerSize;
1601 value = output_[frame_index - 1]->GetContext(); 1601 value = output_[frame_index - 1]->GetContext();
1602 output_frame->SetFrameSlot(output_offset, value); 1602 output_frame->SetFrameSlot(output_offset, value);
1603 DebugPrintOutputSlot(value, frame_index, output_offset, "context\n"); 1603 DebugPrintOutputSlot(value, frame_index, output_offset, "context\n");
1604 1604
1605 // The allocation site.
1606 output_offset -= kPointerSize;
1607 value = reinterpret_cast<intptr_t>(isolate_->heap()->undefined_value());
1608 output_frame->SetFrameSlot(output_offset, value);
1609 DebugPrintOutputSlot(value, frame_index, output_offset, "allocation site\n");
1610
1611 // Number of incoming arguments. 1605 // Number of incoming arguments.
1612 output_offset -= kPointerSize; 1606 output_offset -= kPointerSize;
1613 value = reinterpret_cast<intptr_t>(Smi::FromInt(height - 1)); 1607 value = reinterpret_cast<intptr_t>(Smi::FromInt(height - 1));
1614 output_frame->SetFrameSlot(output_offset, value); 1608 output_frame->SetFrameSlot(output_offset, value);
1615 DebugPrintOutputSlot(value, frame_index, output_offset, "argc "); 1609 DebugPrintOutputSlot(value, frame_index, output_offset, "argc ");
1616 if (trace_scope_ != nullptr) { 1610 if (trace_scope_ != nullptr) {
1617 PrintF(trace_scope_->file(), "(%d)\n", height - 1); 1611 PrintF(trace_scope_->file(), "(%d)\n", height - 1);
1618 } 1612 }
1619 1613
1620 // The newly allocated object was passed as receiver in the artificial 1614 // The newly allocated object was passed as receiver in the artificial
(...skipping 2397 matching lines...) Expand 10 before | Expand all | Expand 10 after
4018 CHECK(value_info->IsMaterializedObject()); 4012 CHECK(value_info->IsMaterializedObject());
4019 4013
4020 value_info->value_ = 4014 value_info->value_ =
4021 Handle<Object>(previously_materialized_objects->get(i), isolate_); 4015 Handle<Object>(previously_materialized_objects->get(i), isolate_);
4022 } 4016 }
4023 } 4017 }
4024 } 4018 }
4025 4019
4026 } // namespace internal 4020 } // namespace internal
4027 } // namespace v8 4021 } // namespace v8
OLDNEW
« no previous file with comments | « src/builtins/x64/builtins-x64.cc ('k') | src/frames.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698