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

Unified Diff: src/compiler/operator.cc

Issue 2425983002: [turbofan] Use uint32 to store the number of control outputs instead of uint16. (Closed)
Patch Set: Increase the maximum number of control outputs instead. Created 4 years, 2 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
« no previous file with comments | « src/compiler/operator.h ('k') | test/mjsunit/regress/wasm/regression-5531.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/operator.cc
diff --git a/src/compiler/operator.cc b/src/compiler/operator.cc
index fa1b2d89ca50106d81e8a44a5812a0b7af2d8fcd..4f746e294491451435444da13be07c1b61938343 100644
--- a/src/compiler/operator.cc
+++ b/src/compiler/operator.cc
@@ -24,7 +24,6 @@ V8_INLINE N CheckRange(size_t val) {
// static
STATIC_CONST_MEMBER_DEFINITION const size_t Operator::kMaxControlOutputCount;
-
Operator::Operator(Opcode opcode, Properties properties, const char* mnemonic,
size_t value_in, size_t effect_in, size_t control_in,
size_t value_out, size_t effect_out, size_t control_out)
@@ -36,8 +35,7 @@ Operator::Operator(Opcode opcode, Properties properties, const char* mnemonic,
control_in_(CheckRange<uint16_t>(control_in)),
value_out_(CheckRange<uint16_t>(value_out)),
effect_out_(CheckRange<uint8_t>(effect_out)),
- control_out_(CheckRange<uint16_t>(control_out)) {}
-
+ control_out_(CheckRange<uint32_t>(control_out)) {}
std::ostream& operator<<(std::ostream& os, const Operator& op) {
op.PrintTo(os);
« no previous file with comments | « src/compiler/operator.h ('k') | test/mjsunit/regress/wasm/regression-5531.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698