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

Unified Diff: runtime/vm/scopes.cc

Issue 2772963003: VM [KERNEL] Avoid emitting :expr_temp and capturing :iterator (Closed)
Patch Set: Created 3 years, 9 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
Index: runtime/vm/scopes.cc
diff --git a/runtime/vm/scopes.cc b/runtime/vm/scopes.cc
index e7c04323367039040209ec798b4ba7870dddae1e..277935bf818100250aba2d9c392e1ad177265982 100644
--- a/runtime/vm/scopes.cc
+++ b/runtime/vm/scopes.cc
@@ -639,7 +639,8 @@ void LocalScope::CaptureLocalVariables(LocalScope* top_scope) {
if (variable->is_forced_stack() ||
(variable->name().raw() == Symbols::StackTraceVar().raw()) ||
(variable->name().raw() == Symbols::ExceptionVar().raw()) ||
- (variable->name().raw() == Symbols::SavedTryContextVar().raw())) {
+ (variable->name().raw() == Symbols::SavedTryContextVar().raw()) ||
+ (variable->name().raw() == Symbols::IteratorParameter().raw())) {
Vyacheslav Egorov (Google) 2017/03/27 14:31:46 Can we move this to kernel_to_il.cc? We prefer to
// Don't capture those variables because the VM expects them to be on
// the stack.
continue;

Powered by Google App Engine
This is Rietveld 408576698