Move closing of generators upon final return to the generator-resume builtin.

test/cctest/interpreter/bytecode_expectations/ForOfLoop.golden

 #
 # Autogenerated by generate-bytecode-expectations.
 #
 
 ---
 wrap: no
 test function name: f
 
 ---
 snippet: "
@@ skipping 625 matching lines @@
 
 ---
 snippet: "
   function* f(arr) {
     for (let x of arr) { let y = x; }
   }
   f([1, 2, 3]);
 "
 frame size: 20
 parameter count: 2
-bytecode array length: 370
+bytecode array length: 358
 bytecodes: [
                 B(Mov), R(new_target), R(11),
                 B(Ldar), R(new_target),
                 B(JumpIfUndefined), U8(25),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetContext), R(11), U8(1),
                 B(PushContext), R(13),
                 B(RestoreGeneratorState), R(11),
                 B(Star), R(12),
                 B(SwitchOnSmiNoFeedback), U8(0), U8(1), I8(0),
                 B(LdaSmi), I8(79),
@@ skipping 18 matching lines @@
   /*   55 S> */ B(Return),
                 B(RestoreGeneratorRegisters), R(11), R(0), U8(14),
                 B(LdaSmi), I8(-2),
                 B(Star), R(12),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetInputOrDebugPos), R(11), U8(1),
                 B(Star), R(14),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetResumeMode), R(11), U8(1),
                 B(SwitchOnSmiNoFeedback), U8(1), U8(2), I8(0),
                 B(Ldar), R(14),
   /*   11 E> */ B(Throw),
-                B(InvokeIntrinsic), U8(Runtime::k_GeneratorClose), R(11), U8(1),
                 B(Ldar), R(14),
   /*   55 S> */ B(Return),
                 B(LdaZero),
                 B(Star), R(7),
                 B(Mov), R(context), R(16),
                 B(Mov), R(context), R(17),
   /*   35 S> */ B(LdaImmutableCurrentContextSlot), U8(4),
                 B(Star), R(18),
                 B(LdaNamedProperty), R(18), U8(3), U8(3),
                 B(Star), R(19),
@@ skipping 91 matching lines @@
                 B(Jump), U8(7),
                 B(CallRuntime), U16(Runtime::kThrowIteratorResultNotAnObject), R(10), U8(1),
                 B(Ldar), R(16),
                 B(SetPendingMessage),
                 B(LdaZero),
                 B(TestEqualStrictNoFeedback), R(14),
                 B(JumpIfFalse), U8(5),
                 B(Ldar), R(15),
                 B(ReThrow),
                 B(LdaUndefined),
-                B(Star), R(14),
-                B(InvokeIntrinsic), U8(Runtime::k_GeneratorClose), R(11), U8(1),
-                B(Ldar), R(14),
   /*   55 S> */ B(Return),
 ]
 constant pool: [
   Smi [53],
-  Smi [14],
+  Smi [10],
   Smi [7],
   SYMBOL_TYPE,
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["next"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["done"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["value"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE [".catch"],
   FIXED_ARRAY_TYPE,
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["return"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE [""],
   FIXED_ARRAY_TYPE,
 ]
 handlers: [
-  [108, 226, 232],
-  [111, 190, 192],
-  [292, 302, 304],
+  [104, 222, 228],
+  [107, 186, 188],
+  [288, 298, 300],
 ]
 
 ---
 snippet: "
   function* f(arr) {
     for (let x of arr) yield x;
   }
   f([1, 2, 3]);
 "
 frame size: 20
 parameter count: 2
-bytecode array length: 453
+bytecode array length: 437
 bytecodes: [
                 B(Mov), R(new_target), R(10),
                 B(Ldar), R(new_target),
                 B(JumpIfUndefined), U8(25),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetContext), R(10), U8(1),
                 B(PushContext), R(12),
                 B(RestoreGeneratorState), R(10),
                 B(Star), R(11),
                 B(SwitchOnSmiNoFeedback), U8(0), U8(2), I8(0),
                 B(LdaSmi), I8(79),
@@ skipping 18 matching lines @@
   /*   49 S> */ B(Return),
                 B(RestoreGeneratorRegisters), R(10), R(0), U8(13),
                 B(LdaSmi), I8(-2),
                 B(Star), R(11),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetInputOrDebugPos), R(10), U8(1),
                 B(Star), R(13),
                 B(InvokeIntrinsic), U8(Runtime::k_GeneratorGetResumeMode), R(10), U8(1),
                 B(SwitchOnSmiNoFeedback), U8(2), U8(2), I8(0),
                 B(Ldar), R(13),
   /*   11 E> */ B(Throw),
-                B(InvokeIntrinsic), U8(Runtime::k_GeneratorClose), R(10), U8(1),
                 B(Ldar), R(13),
   /*   49 S> */ B(Return),
                 B(LdaZero),
                 B(Star), R(6),
                 B(Mov), R(context), R(15),
                 B(Mov), R(context), R(16),
   /*   35 S> */ B(LdaImmutableCurrentContextSlot), U8(4),
                 B(Star), R(17),
                 B(LdaNamedProperty), R(17), U8(4), U8(3),
                 B(Star), R(18),
@@ skipping 115 matching lines @@
                 B(InvokeIntrinsic), U8(Runtime::k_Call), R(16), U8(2),
                 B(Star), R(9),
                 B(InvokeIntrinsic), U8(Runtime::k_IsJSReceiver), R(9), U8(1),
                 B(JumpIfToBooleanFalse), U8(4),
                 B(Jump), U8(7),
                 B(CallRuntime), U16(Runtime::kThrowIteratorResultNotAnObject), R(9), U8(1),
                 B(Ldar), R(15),
                 B(SetPendingMessage),
                 B(Ldar), R(13),
                 B(SwitchOnSmiNoFeedback), U8(16), U8(2), I8(0),
-                B(Jump), U8(12),
-                B(InvokeIntrinsic), U8(Runtime::k_GeneratorClose), R(10), U8(1),
+                B(Jump), U8(8),
                 B(Ldar), R(14),
   /*   49 S> */ B(Return),
                 B(Ldar), R(14),
                 B(ReThrow),
                 B(LdaUndefined),
-                B(Star), R(13),
-                B(InvokeIntrinsic), U8(Runtime::k_GeneratorClose), R(10), U8(1),
-                B(Ldar), R(13),
   /*   49 S> */ B(Return),
 ]
 constant pool: [
   Smi [53],
-  Smi [117],
-  Smi [14],
+  Smi [113],
+  Smi [10],
   Smi [7],
   SYMBOL_TYPE,
   Smi [87],
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["next"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["done"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["value"],
   Smi [15],
   Smi [7],
   ONE_BYTE_INTERNALIZED_STRING_TYPE [".catch"],
   FIXED_ARRAY_TYPE,
   ONE_BYTE_INTERNALIZED_STRING_TYPE ["return"],
   ONE_BYTE_INTERNALIZED_STRING_TYPE [""],
   FIXED_ARRAY_TYPE,
   Smi [6],
-  Smi [13],
+  Smi [9],
 ]
 handlers: [
-  [108, 298, 304],
-  [111, 262, 264],
-  [365, 375, 377],
+  [104, 294, 300],
+  [107, 258, 260],
+  [361, 371, 373],
 ]
 
 ---
 snippet: "
   async function f(arr) {
     for (let x of arr) { let y = x; }
   }
   f([1, 2, 3]);
 "
 frame size: 23
@@ skipping 465 matching lines @@
   Smi [9],
 ]
 handlers: [
   [70, 492, 498],
   [73, 447, 449],
   [79, 280, 286],
   [82, 240, 242],
   [347, 357, 359],
 ]