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

Side by Side Diff: src/v8globals.h

Issue 13408005: Force context allocation for variables in generator scopes. (Closed) Base URL: git://github.com/v8/v8.git@master
Patch Set: Created 7 years, 8 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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 478 matching lines...) Expand 10 before | Expand all | Expand 10 after
489 LET, // declared via 'let' declarations (first lexical) 489 LET, // declared via 'let' declarations (first lexical)
490 490
491 CONST_HARMONY, // declared via 'const' declarations in harmony mode 491 CONST_HARMONY, // declared via 'const' declarations in harmony mode
492 492
493 MODULE, // declared via 'module' declaration (last lexical) 493 MODULE, // declared via 'module' declaration (last lexical)
494 494
495 // Variables introduced by the compiler: 495 // Variables introduced by the compiler:
496 INTERNAL, // like VAR, but not user-visible (may or may not 496 INTERNAL, // like VAR, but not user-visible (may or may not
497 // be in a context) 497 // be in a context)
498 498
499 TEMPORARY, // temporary variables (not user-visible), never 499 TEMPORARY, // temporary variables (not user-visible), usually
500 // in a context 500 // stack-allocated
Michael Starzinger 2013/04/05 11:38:42 nit: Let's add "... (unless the scope forces conte
wingo 2013/04/05 12:25:06 Done.
501 501
502 DYNAMIC, // always require dynamic lookup (we don't know 502 DYNAMIC, // always require dynamic lookup (we don't know
503 // the declaration) 503 // the declaration)
504 504
505 DYNAMIC_GLOBAL, // requires dynamic lookup, but we know that the 505 DYNAMIC_GLOBAL, // requires dynamic lookup, but we know that the
506 // variable is global unless it has been shadowed 506 // variable is global unless it has been shadowed
507 // by an eval-introduced variable 507 // by an eval-introduced variable
508 508
509 DYNAMIC_LOCAL // requires dynamic lookup, but we know that the 509 DYNAMIC_LOCAL // requires dynamic lookup, but we know that the
510 // variable is local and where it is unless it 510 // variable is local and where it is unless it
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
572 572
573 enum ClearExceptionFlag { 573 enum ClearExceptionFlag {
574 KEEP_EXCEPTION, 574 KEEP_EXCEPTION,
575 CLEAR_EXCEPTION 575 CLEAR_EXCEPTION
576 }; 576 };
577 577
578 578
579 } } // namespace v8::internal 579 } } // namespace v8::internal
580 580
581 #endif // V8_V8GLOBALS_H_ 581 #endif // V8_V8GLOBALS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698