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

Side by Side Diff: src/compilation-dependencies.cc

Issue 2067423003: [turbofan] Properly handle dictionary maps in the prototype chain. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@TurboFan_CheckedOperators
Patch Set: Created 4 years, 6 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/compilation-dependencies.h ('k') | src/compiler/js-native-context-specialization.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 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 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/compilation-dependencies.h" 5 #include "src/compilation-dependencies.h"
6 6
7 #include "src/factory.h" 7 #include "src/factory.h"
8 #include "src/handles-inl.h" 8 #include "src/handles-inl.h"
9 #include "src/isolate.h" 9 #include "src/isolate.h"
10 #include "src/objects-inl.h" 10 #include "src/objects-inl.h"
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 117
118 void CompilationDependencies::AssumeMapStable(Handle<Map> map) { 118 void CompilationDependencies::AssumeMapStable(Handle<Map> map) {
119 DCHECK(map->is_stable()); 119 DCHECK(map->is_stable());
120 // Do nothing if the map cannot transition. 120 // Do nothing if the map cannot transition.
121 if (map->CanTransition()) { 121 if (map->CanTransition()) {
122 Insert(DependentCode::kPrototypeCheckGroup, map); 122 Insert(DependentCode::kPrototypeCheckGroup, map);
123 } 123 }
124 } 124 }
125 125
126 126
127 void CompilationDependencies::AssumePrototypeMapsStable(
128 Handle<Map> map, MaybeHandle<JSReceiver> prototype) {
129 for (PrototypeIterator i(map); !i.IsAtEnd(); i.Advance()) {
130 Handle<JSReceiver> const current =
131 PrototypeIterator::GetCurrent<JSReceiver>(i);
132 AssumeMapStable(handle(current->map()));
133 Handle<JSReceiver> last;
134 if (prototype.ToHandle(&last) && last.is_identical_to(current)) {
135 break;
136 }
137 }
138 }
139
140
141 void CompilationDependencies::AssumeTransitionStable( 127 void CompilationDependencies::AssumeTransitionStable(
142 Handle<AllocationSite> site) { 128 Handle<AllocationSite> site) {
143 // Do nothing if the object doesn't have any useful element transitions left. 129 // Do nothing if the object doesn't have any useful element transitions left.
144 ElementsKind kind = 130 ElementsKind kind =
145 site->SitePointsToLiteral() 131 site->SitePointsToLiteral()
146 ? JSObject::cast(site->transition_info())->GetElementsKind() 132 ? JSObject::cast(site->transition_info())->GetElementsKind()
147 : site->GetElementsKind(); 133 : site->GetElementsKind();
148 if (AllocationSite::GetMode(kind) == TRACK_ALLOCATION_SITE) { 134 if (AllocationSite::GetMode(kind) == TRACK_ALLOCATION_SITE) {
149 Insert(DependentCode::kAllocationSiteTransitionChangedGroup, site); 135 Insert(DependentCode::kAllocationSiteTransitionChangedGroup, site);
150 } 136 }
151 } 137 }
152 138
153 } // namespace internal 139 } // namespace internal
154 } // namespace v8 140 } // namespace v8
OLDNEW
« no previous file with comments | « src/compilation-dependencies.h ('k') | src/compiler/js-native-context-specialization.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698