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

Side by Side Diff: src/compiler/js-operator.cc

Issue 2541813002: [compiler] Remove context value input from JSLoadContext and JSStoreContext. (Closed)
Patch Set: Rebase. Created 4 years 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/js-global-object-specialization.cc ('k') | src/compiler/js-typed-lowering.cc » ('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-operator.h" 5 #include "src/compiler/js-operator.h"
6 6
7 #include <limits> 7 #include <limits>
8 8
9 #include "src/base/lazy-instance.h" 9 #include "src/base/lazy-instance.h"
10 #include "src/compiler/opcodes.h" 10 #include "src/compiler/opcodes.h"
(...skipping 738 matching lines...) Expand 10 before | Expand all | Expand 10 after
749 } 749 }
750 750
751 751
752 const Operator* JSOperatorBuilder::LoadContext(size_t depth, size_t index, 752 const Operator* JSOperatorBuilder::LoadContext(size_t depth, size_t index,
753 bool immutable) { 753 bool immutable) {
754 ContextAccess access(depth, index, immutable); 754 ContextAccess access(depth, index, immutable);
755 return new (zone()) Operator1<ContextAccess>( // -- 755 return new (zone()) Operator1<ContextAccess>( // --
756 IrOpcode::kJSLoadContext, // opcode 756 IrOpcode::kJSLoadContext, // opcode
757 Operator::kNoWrite | Operator::kNoThrow, // flags 757 Operator::kNoWrite | Operator::kNoThrow, // flags
758 "JSLoadContext", // name 758 "JSLoadContext", // name
759 1, 1, 0, 1, 1, 0, // counts 759 0, 1, 0, 1, 1, 0, // counts
760 access); // parameter 760 access); // parameter
761 } 761 }
762 762
763 763
764 const Operator* JSOperatorBuilder::StoreContext(size_t depth, size_t index) { 764 const Operator* JSOperatorBuilder::StoreContext(size_t depth, size_t index) {
765 ContextAccess access(depth, index, false); 765 ContextAccess access(depth, index, false);
766 return new (zone()) Operator1<ContextAccess>( // -- 766 return new (zone()) Operator1<ContextAccess>( // --
767 IrOpcode::kJSStoreContext, // opcode 767 IrOpcode::kJSStoreContext, // opcode
768 Operator::kNoRead | Operator::kNoThrow, // flags 768 Operator::kNoRead | Operator::kNoThrow, // flags
769 "JSStoreContext", // name 769 "JSStoreContext", // name
770 2, 1, 1, 0, 1, 0, // counts 770 1, 1, 1, 0, 1, 0, // counts
771 access); // parameter 771 access); // parameter
772 } 772 }
773 773
774 const Operator* JSOperatorBuilder::LoadModule(int32_t cell_index) { 774 const Operator* JSOperatorBuilder::LoadModule(int32_t cell_index) {
775 return new (zone()) Operator1<int32_t>( // -- 775 return new (zone()) Operator1<int32_t>( // --
776 IrOpcode::kJSLoadModule, // opcode 776 IrOpcode::kJSLoadModule, // opcode
777 Operator::kNoWrite | Operator::kNoThrow, // flags 777 Operator::kNoWrite | Operator::kNoThrow, // flags
778 "JSLoadModule", // name 778 "JSLoadModule", // name
779 1, 1, 1, 1, 1, 0, // counts 779 1, 1, 1, 1, 1, 0, // counts
780 cell_index); // parameter 780 cell_index); // parameter
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
900 return new (zone()) Operator1<Handle<ScopeInfo>>( // -- 900 return new (zone()) Operator1<Handle<ScopeInfo>>( // --
901 IrOpcode::kJSCreateScriptContext, Operator::kNoProperties, // opcode 901 IrOpcode::kJSCreateScriptContext, Operator::kNoProperties, // opcode
902 "JSCreateScriptContext", // name 902 "JSCreateScriptContext", // name
903 1, 1, 1, 1, 1, 2, // counts 903 1, 1, 1, 1, 1, 2, // counts
904 scpope_info); // parameter 904 scpope_info); // parameter
905 } 905 }
906 906
907 } // namespace compiler 907 } // namespace compiler
908 } // namespace internal 908 } // namespace internal
909 } // namespace v8 909 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/js-global-object-specialization.cc ('k') | src/compiler/js-typed-lowering.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698