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

Side by Side Diff: src/scopeinfo.h

Issue 11093074: Get rid of static module allocation, do it in code. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Addressed last comments; added other back-ends Created 8 years, 1 month 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 | Annotate | Revision Log
« no previous file with comments | « src/runtime.cc ('k') | src/scopeinfo.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 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 VariableMode mode() { return ModeField::decode(value_); } 107 VariableMode mode() { return ModeField::decode(value_); }
108 108
109 InitializationFlag initialization_flag() { 109 InitializationFlag initialization_flag() {
110 return InitField::decode(value_); 110 return InitField::decode(value_);
111 } 111 }
112 112
113 int index() { return IndexField::decode(value_); } 113 int index() { return IndexField::decode(value_); }
114 114
115 // Bit fields in value_ (type, shift, size). Must be public so the 115 // Bit fields in value_ (type, shift, size). Must be public so the
116 // constants can be embedded in generated code. 116 // constants can be embedded in generated code.
117 class ModeField: public BitField<VariableMode, 0, 3> {}; 117 class ModeField: public BitField<VariableMode, 0, 4> {};
118 class InitField: public BitField<InitializationFlag, 3, 1> {}; 118 class InitField: public BitField<InitializationFlag, 4, 1> {};
119 class IndexField: public BitField<int, 4, 32-4> {}; 119 class IndexField: public BitField<int, 5, 32-5> {};
120 120
121 private: 121 private:
122 uint32_t value_; 122 uint32_t value_;
123 }; 123 };
124 124
125 Key keys_[kLength]; 125 Key keys_[kLength];
126 uint32_t values_[kLength]; 126 uint32_t values_[kLength];
127 127
128 friend class Isolate; 128 friend class Isolate;
129 DISALLOW_COPY_AND_ASSIGN(ContextSlotCache); 129 DISALLOW_COPY_AND_ASSIGN(ContextSlotCache);
130 }; 130 };
131 131
132 132
133
134
135 //---------------------------------------------------------------------------
136 // Auxiliary class used for the description of module instances.
137 // Used by Runtime_DeclareModules.
138
139 class ModuleInfo: public FixedArray {
140 public:
141 static ModuleInfo* cast(Object* description) {
142 return static_cast<ModuleInfo*>(FixedArray::cast(description));
143 }
144
145 static Handle<ModuleInfo> Create(
146 Isolate* isolate, Interface* interface, Scope* scope);
147
148 // Index of module's context in host context.
149 int host_index() { return Smi::cast(get(HOST_OFFSET))->value(); }
150
151 // Name, mode, and index of the i-th export, respectively.
152 // For value exports, the index is the slot of the value in the module
153 // context, for exported modules it is the slot index of the
154 // referred module's context in the host context.
155 // TODO(rossberg): This format cannot yet handle exports of modules declared
156 // in earlier scripts.
157 String* name(int i) { return String::cast(get(name_offset(i))); }
158 VariableMode mode(int i) {
159 return static_cast<VariableMode>(Smi::cast(get(mode_offset(i)))->value());
160 }
161 int index(int i) { return Smi::cast(get(index_offset(i)))->value(); }
162
163 int length() { return (FixedArray::length() - HEADER_SIZE) / ITEM_SIZE; }
164
165 private:
166 // The internal format is: Index, (Name, VariableMode, Index)*
167 enum {
168 HOST_OFFSET,
169 NAME_OFFSET,
170 MODE_OFFSET,
171 INDEX_OFFSET,
172 HEADER_SIZE = NAME_OFFSET,
173 ITEM_SIZE = INDEX_OFFSET - NAME_OFFSET + 1
174 };
175 inline int name_offset(int i) { return NAME_OFFSET + i * ITEM_SIZE; }
176 inline int mode_offset(int i) { return MODE_OFFSET + i * ITEM_SIZE; }
177 inline int index_offset(int i) { return INDEX_OFFSET + i * ITEM_SIZE; }
178
179 static Handle<ModuleInfo> Allocate(Isolate* isolate, int length) {
180 return Handle<ModuleInfo>::cast(
181 isolate->factory()->NewFixedArray(HEADER_SIZE + ITEM_SIZE * length));
182 }
183 void set_host_index(int index) { set(HOST_OFFSET, Smi::FromInt(index)); }
184 void set_name(int i, String* name) { set(name_offset(i), name); }
185 void set_mode(int i, VariableMode mode) {
186 set(mode_offset(i), Smi::FromInt(mode));
187 }
188 void set_index(int i, int index) {
189 set(index_offset(i), Smi::FromInt(index));
190 }
191 };
192
193
133 } } // namespace v8::internal 194 } } // namespace v8::internal
134 195
135 #endif // V8_SCOPEINFO_H_ 196 #endif // V8_SCOPEINFO_H_
OLDNEW
« no previous file with comments | « src/runtime.cc ('k') | src/scopeinfo.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698