"Runtime Semantics: EvaluateBody" seems inconsistent with the existing spec

[domenic]

As far as I can tell, this spec's step 1 (performing FunctionDeclarationInstantiation) is handled in the ES spec by OrdinaryCallEvaluateBody, so it shouldn't be in the runtime semantics here.

[zenparsing]

Thanks for this great review work!

I was basing the EvaluateBody steps on the async-await spec, which does some refactoring there:

https://tc39.github.io/ecmascript-asyncawait/#sec-modified-OrdinaryCallEvaluateBody

[domenic]

Confirmed this is how it should work, and even managed to find a bug in tc39/ecma262#692 while doing so (it didn't carry over the update to OrdinaryCallEvaluateBody). Closing!