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

Side by Side Diff: src/ast/variables.cc

Issue 2950993002: Make some functions that are hit during renderer startup available for inlining (Closed)
Patch Set: Created 3 years, 6 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
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 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/ast/variables.h" 5 #include "src/ast/variables.h"
6 6
7 #include "src/ast/scopes.h" 7 #include "src/ast/scopes.h"
8 #include "src/globals.h" 8 #include "src/globals.h"
9 #include "src/objects-inl.h" 9 #include "src/objects-inl.h"
10 10
11 namespace v8 { 11 namespace v8 {
12 namespace internal { 12 namespace internal {
13 13
14 // ---------------------------------------------------------------------------- 14 // ----------------------------------------------------------------------------
15 // Implementation Variable. 15 // Implementation Variable.
16 16
17 Variable::Variable(Scope* scope, const AstRawString* name, VariableMode mode,
18 VariableKind kind, InitializationFlag initialization_flag,
19 MaybeAssignedFlag maybe_assigned_flag)
20 : scope_(scope),
21 name_(name),
22 local_if_not_shadowed_(nullptr),
23 next_(nullptr),
24 index_(-1),
25 initializer_position_(kNoSourcePosition),
26 bit_field_(MaybeAssignedFlagField::encode(maybe_assigned_flag) |
27 InitializationFlagField::encode(initialization_flag) |
28 VariableModeField::encode(mode) | IsUsedField::encode(false) |
29 ForceContextAllocationField::encode(false) |
30 ForceHoleInitializationField::encode(false) |
31 LocationField::encode(VariableLocation::UNALLOCATED) |
32 VariableKindField::encode(kind)) {
33 // Var declared variables never need initialization.
34 DCHECK(!(mode == VAR && initialization_flag == kNeedsInitialization));
35 }
36
37
38 bool Variable::IsGlobalObjectProperty() const { 17 bool Variable::IsGlobalObjectProperty() const {
39 // Temporaries are never global, they must always be allocated in the 18 // Temporaries are never global, they must always be allocated in the
40 // activation frame. 19 // activation frame.
41 return (IsDynamicVariableMode(mode()) || mode() == VAR) && 20 return (IsDynamicVariableMode(mode()) || mode() == VAR) &&
42 scope_ != nullptr && scope_->is_script_scope(); 21 scope_ != nullptr && scope_->is_script_scope();
43 } 22 }
44 23
45 } // namespace internal 24 } // namespace internal
46 } // namespace v8 25 } // namespace v8
OLDNEW
« no previous file with comments | « src/ast/variables.h ('k') | src/heap/heap.h » ('j') | src/interpreter/bytecodes.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698