| 19:26 | <bakkot> | at the editor call we talked about https://github.com/tc39/ecma262/pull/3606 |
| 19:27 | <bakkot> | and making it so that built-in generators/async functions always return NormalCompletion(*undefined*) |
| 19:27 | <bakkot> | I still think this is the right thing to do |
| 19:28 | <bakkot> | however, the PR updates the machinery to assert that they never return a return completion, and this is not true for the generator case because .return can inject a return completion wherever we use the Yield macro |
| 19:28 | <bakkot> | this sucks and I hate it |
| 19:30 | <bakkot> | options: restore the machinery so that built-in generators can handle all three kinds of completion, update our use of the Yield macro to explicitly handle return completions wherever it's used, or switch to the other PR which made them all return ReturnCompletions instead and then not handle the NormalCompletion case |
| 19:30 | <bakkot> | my preference is the first but I could be persuaded that the second is better |
| 19:30 | <bakkot> | second ends up being a fair bit of boilerplate though |