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

Side by Side Diff: src/compiler/js-generic-lowering.cc

Issue 2587393006: [runtime] Collect IC feedback in DefineDataPropertyInLiteral. (Closed)
Patch Set: Rebase. Created 3 years, 11 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/compiler/code-assembler.cc ('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 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 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/compiler/js-generic-lowering.h" 5 #include "src/compiler/js-generic-lowering.h"
6 6
7 #include "src/ast/ast.h" 7 #include "src/ast/ast.h"
8 #include "src/builtins/builtins-constructor.h" 8 #include "src/builtins/builtins-constructor.h"
9 #include "src/code-factory.h" 9 #include "src/code-factory.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 node->ReplaceInput(Descriptor::kName, jsgraph()->HeapConstant(p.name())); 257 node->ReplaceInput(Descriptor::kName, jsgraph()->HeapConstant(p.name()));
258 node->ReplaceInput(Descriptor::kValue, value); 258 node->ReplaceInput(Descriptor::kValue, value);
259 node->ReplaceInput(Descriptor::kSlot, 259 node->ReplaceInput(Descriptor::kSlot,
260 jsgraph()->SmiConstant(p.feedback().index())); 260 jsgraph()->SmiConstant(p.feedback().index()));
261 node->ReplaceInput(Descriptor::kVector, vector); 261 node->ReplaceInput(Descriptor::kVector, vector);
262 node->ReplaceInput(7, effect); 262 node->ReplaceInput(7, effect);
263 ReplaceWithStubCall(node, callable, flags); 263 ReplaceWithStubCall(node, callable, flags);
264 } 264 }
265 265
266 void JSGenericLowering::LowerJSStoreDataPropertyInLiteral(Node* node) { 266 void JSGenericLowering::LowerJSStoreDataPropertyInLiteral(Node* node) {
267 DataPropertyParameters const& p = DataPropertyParametersOf(node->op());
268 node->InsertInputs(zone(), 4, 2);
269 node->ReplaceInput(4, jsgraph()->HeapConstant(p.feedback().vector()));
270 node->ReplaceInput(5, jsgraph()->SmiConstant(p.feedback().index()));
267 ReplaceWithRuntimeCall(node, Runtime::kDefineDataPropertyInLiteral); 271 ReplaceWithRuntimeCall(node, Runtime::kDefineDataPropertyInLiteral);
268 } 272 }
269 273
270 void JSGenericLowering::LowerJSDeleteProperty(Node* node) { 274 void JSGenericLowering::LowerJSDeleteProperty(Node* node) {
271 LanguageMode language_mode = OpParameter<LanguageMode>(node); 275 LanguageMode language_mode = OpParameter<LanguageMode>(node);
272 ReplaceWithRuntimeCall(node, is_strict(language_mode) 276 ReplaceWithRuntimeCall(node, is_strict(language_mode)
273 ? Runtime::kDeleteProperty_Strict 277 ? Runtime::kDeleteProperty_Strict
274 : Runtime::kDeleteProperty_Sloppy); 278 : Runtime::kDeleteProperty_Sloppy);
275 } 279 }
276 280
(...skipping 356 matching lines...) Expand 10 before | Expand all | Expand 10 after
633 } 637 }
634 638
635 639
636 MachineOperatorBuilder* JSGenericLowering::machine() const { 640 MachineOperatorBuilder* JSGenericLowering::machine() const {
637 return jsgraph()->machine(); 641 return jsgraph()->machine();
638 } 642 }
639 643
640 } // namespace compiler 644 } // namespace compiler
641 } // namespace internal 645 } // namespace internal
642 } // namespace v8 646 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/code-assembler.cc ('k') | src/compiler/js-native-context-specialization.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698