Force context allocation for variables in generator scopes.

src/ia32/full-codegen-ia32.cc

Index: src/ia32/full-codegen-ia32.cc
diff --git a/src/ia32/full-codegen-ia32.cc b/src/ia32/full-codegen-ia32.cc
index efbdf135445b9988494af3affdfab5775f3bd9f4..bce4805a5170a677e8daca909117e005ae4069cf 100644
--- a/src/ia32/full-codegen-ia32.cc
+++ b/src/ia32/full-codegen-ia32.cc
@@ -164,6 +164,8 @@ void FullCodeGenerator::Generate() {
 
   { Comment cmnt(masm_, "[ Allocate locals");
     int locals_count = info->scope()->num_stack_slots();
+    // Generators allocate locals, if any, in context slots.
+    ASSERT(!(info->function()->is_generator() && locals_count));

[Michael Starzinger, 2013/04/05 11:38:42]

nit: Can we rewrite this as "!info->function()->is_generator() || locals_count
== 0", I think that's easier to read. This also applies to the other
architectures.

[wingo, 2013/04/05 12:25:06]

Done.

     if (locals_count == 1) {
       __ push(Immediate(isolate()->factory()->undefined_value()));
     } else if (locals_count > 1) {