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

Side by Side Diff: src/scopes.h

Issue 114024: Avoid linear search for non-locals in the scope code... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 11 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | src/scopes.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 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 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 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 Handle<String> scope_name_; 271 Handle<String> scope_name_;
272 272
273 // The variables declared in this scope: 273 // The variables declared in this scope:
274 // all user-declared variables (incl. parameters) 274 // all user-declared variables (incl. parameters)
275 LocalsMap locals_; 275 LocalsMap locals_;
276 // compiler-allocated (user-invisible) temporaries 276 // compiler-allocated (user-invisible) temporaries
277 ZoneList<Variable*> temps_; 277 ZoneList<Variable*> temps_;
278 // parameter list in source order 278 // parameter list in source order
279 ZoneList<Variable*> params_; 279 ZoneList<Variable*> params_;
280 // variables that must be looked up dynamically 280 // variables that must be looked up dynamically
281 ZoneList<Variable*> nonlocals_; 281 LocalsMap dynamics_;
282 LocalsMap dynamics_local_;
283 LocalsMap dynamics_global_;
282 // unresolved variables referred to from this scope 284 // unresolved variables referred to from this scope
283 ZoneList<VariableProxy*> unresolved_; 285 ZoneList<VariableProxy*> unresolved_;
284 // declarations 286 // declarations
285 ZoneList<Declaration*> decls_; 287 ZoneList<Declaration*> decls_;
286 // convenience variable 288 // convenience variable
287 VariableProxy* receiver_; 289 VariableProxy* receiver_;
288 // function variable, if any; function scopes only 290 // function variable, if any; function scopes only
289 Variable* function_; 291 Variable* function_;
290 // convenience variable; function scopes only 292 // convenience variable; function scopes only
291 VariableProxy* arguments_; 293 VariableProxy* arguments_;
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
357 virtual VariableProxy* NewUnresolved(Handle<String> name, bool inside_with) { 359 virtual VariableProxy* NewUnresolved(Handle<String> name, bool inside_with) {
358 return NULL; 360 return NULL;
359 } 361 }
360 virtual VariableProxy* NewTemporary(Handle<String> name) { return NULL; } 362 virtual VariableProxy* NewTemporary(Handle<String> name) { return NULL; }
361 }; 363 };
362 364
363 365
364 } } // namespace v8::internal 366 } } // namespace v8::internal
365 367
366 #endif // V8_SCOPES_H_ 368 #endif // V8_SCOPES_H_
OLDNEW
« no previous file with comments | « no previous file | src/scopes.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698