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

Side by Side Diff: src/compiler/instruction.h

Issue 2572683005: [turbofan] Reserve the StateValueList's size before filling it (Closed)
Patch Set: 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 | « no previous file | src/compiler/instruction-selector.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 #ifndef V8_COMPILER_INSTRUCTION_H_ 5 #ifndef V8_COMPILER_INSTRUCTION_H_
6 #define V8_COMPILER_INSTRUCTION_H_ 6 #define V8_COMPILER_INSTRUCTION_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <iosfwd> 9 #include <iosfwd>
10 #include <map> 10 #include <map>
(...skipping 1138 matching lines...) Expand 10 before | Expand all | Expand 10 after
1149 MachineType type_; 1149 MachineType type_;
1150 size_t id_; 1150 size_t id_;
1151 }; 1151 };
1152 1152
1153 class StateValueList { 1153 class StateValueList {
1154 public: 1154 public:
1155 explicit StateValueList(Zone* zone) : fields_(zone), nested_(zone) {} 1155 explicit StateValueList(Zone* zone) : fields_(zone), nested_(zone) {}
1156 1156
1157 size_t size() { return fields_.size(); } 1157 size_t size() { return fields_.size(); }
1158 1158
1159 void reserve(size_t size) { fields_.reserve(size); }
Jarin 2016/12/14 14:26:28 style&naming nit: reserve -> ReserveSize
1160
1159 struct Value { 1161 struct Value {
1160 StateValueDescriptor* desc; 1162 StateValueDescriptor* desc;
1161 StateValueList* nested; 1163 StateValueList* nested;
1162 1164
1163 Value(StateValueDescriptor* desc, StateValueList* nested) 1165 Value(StateValueDescriptor* desc, StateValueList* nested)
1164 : desc(desc), nested(nested) {} 1166 : desc(desc), nested(nested) {}
1165 }; 1167 };
1166 1168
1167 class iterator { 1169 class iterator {
1168 public: 1170 public:
(...skipping 433 matching lines...) Expand 10 before | Expand all | Expand 10 after
1602 }; 1604 };
1603 1605
1604 V8_EXPORT_PRIVATE std::ostream& operator<<( 1606 V8_EXPORT_PRIVATE std::ostream& operator<<(
1605 std::ostream& os, const PrintableInstructionSequence& code); 1607 std::ostream& os, const PrintableInstructionSequence& code);
1606 1608
1607 } // namespace compiler 1609 } // namespace compiler
1608 } // namespace internal 1610 } // namespace internal
1609 } // namespace v8 1611 } // namespace v8
1610 1612
1611 #endif // V8_COMPILER_INSTRUCTION_H_ 1613 #endif // V8_COMPILER_INSTRUCTION_H_
OLDNEW
« no previous file with comments | « no previous file | src/compiler/instruction-selector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698