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

Unified Diff: runtime/vm/stack_frame.cc

Issue 1756403002: VM: Add smi fast path operations for precompiled code (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: simplified register constraints on arm, arm64 Created 4 years, 10 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
Index: runtime/vm/stack_frame.cc
diff --git a/runtime/vm/stack_frame.cc b/runtime/vm/stack_frame.cc
index fb3d8d858cc0358169d7f1746ad3bcff87891e0d..d7016728a836155e6159693b78129aa12eea94f6 100644
--- a/runtime/vm/stack_frame.cc
+++ b/runtime/vm/stack_frame.cc
@@ -112,16 +112,12 @@ void StackFrame::VisitObjectPointers(ObjectPointerVisitor* visitor) {
// visit frame slots which are marked as having objects.
//
// The layout of the frame is (lower addresses to the right):
- // | spill slots | outgoing arguments | saved registers |
- // |XXXXXXXXXXXXX|--------------------|XXXXXXXXXXXXXXXXX|
+ // | spill slots | outgoing arguments | saved registers | slow-path args |
+ // |XXXXXXXXXXXXX|--------------------|XXXXXXXXXXXXXXXXX|XXXXXXXXXXXXXXXX|
//
// The spill slots and any saved registers are described in the stack
// map. The outgoing arguments are assumed to be tagged; the number
// of outgoing arguments is not explicitly tracked.
- //
- // TODO(kmillikin): This does not handle slow path calls with
Vyacheslav Egorov (Google) 2016/03/04 13:30:36 Hooray!
- // arguments, where the arguments are pushed after the live registers.
- // Enable such calls.
intptr_t length = map.Length();
// Spill slots are at the 'bottom' of the frame.
intptr_t spill_slot_count = length - map.RegisterBitCount();
« runtime/vm/intermediate_language_x64.cc ('K') | « runtime/vm/intermediate_language_x64.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698