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

Unified Diff: src/globals.h

Issue 2249203002: Revert of Better pack fields in Variable (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 4 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/ast/variables.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/globals.h
diff --git a/src/globals.h b/src/globals.h
index d9c02ffc03f457ccbe2775162c3f6ac13e43289c..6fc93d92db2707cc09b0d3ef6f27141513220aa6 100644
--- a/src/globals.h
+++ b/src/globals.h
@@ -878,7 +878,7 @@
// The order of this enum has to be kept in sync with the predicates below.
-enum VariableMode : uint8_t {
+enum VariableMode {
// User declared variables:
VAR, // declared via 'var', and 'function' declarations
@@ -899,12 +899,10 @@
// variable is global unless it has been shadowed
// by an eval-introduced variable
- DYNAMIC_LOCAL, // requires dynamic lookup, but we know that the
- // variable is local and where it is unless it
- // has been shadowed by an eval-introduced
- // variable
-
- kLastVariableMode = DYNAMIC_LOCAL
+ DYNAMIC_LOCAL // requires dynamic lookup, but we know that the
+ // variable is local and where it is unless it
+ // has been shadowed by an eval-introduced
+ // variable
};
inline bool IsDynamicVariableMode(VariableMode mode) {
@@ -926,7 +924,7 @@
return mode == CONST || mode == CONST_LEGACY;
}
-enum class VariableLocation : uint8_t {
+enum class VariableLocation {
// Before and during variable allocation, a variable whose location is
// not yet determined. After allocation, a variable looked up as a
// property on the global object (and possibly absent). name() is the
@@ -959,9 +957,7 @@
LOOKUP,
// A named slot in a module's export table.
- MODULE,
-
- kLastVariableLocation = MODULE
+ MODULE
};
// ES6 Draft Rev3 10.2 specifies declarative environment records with mutable
@@ -995,9 +991,14 @@
// The following enum specifies a flag that indicates if the binding needs a
// distinct initialization step (kNeedsInitialization) or if the binding is
// immediately initialized upon creation (kCreatedInitialized).
-enum InitializationFlag : uint8_t { kNeedsInitialization, kCreatedInitialized };
-
-enum MaybeAssignedFlag : uint8_t { kNotAssigned, kMaybeAssigned };
+enum InitializationFlag {
+ kNeedsInitialization,
+ kCreatedInitialized
+};
+
+
+enum MaybeAssignedFlag { kNotAssigned, kMaybeAssigned };
+
// Serialized in PreparseData, so numeric values should not be changed.
enum ParseErrorType { kSyntaxError = 0, kReferenceError = 1 };
« no previous file with comments | « src/ast/variables.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698