polish: remove misleading comment

yaacovCR · yaacovCR · commit afd722efd40d · 2022-12-27T15:31:12.000+02:00
We could not demonstrate via actual benchmarking that the tick reduction from awaiting a promise prior to returning it translates into real-world performance gains. See #3796. In any case, the promise must be awaited (irrespective of performance) to allow the catch block to handle rejection errors. Simply returning `completed` would result in test failures.
diff --git a/src/execution/execute.ts b/src/execution/execute.ts
@@ -933,8 +933,6 @@ async function completePromisedValue(
       asyncPayloadRecord,
     );
     if (isPromise(completed)) {
-      // see: https://github.com/tc39/proposal-faster-promise-adoption
-      // it is faster to await a promise prior to returning it from an async function
       completed = await completed;
     }
     return completed;

Original file line number	Diff line number	Diff line change
`@@ -933,8 +933,6 @@ async function completePromisedValue(`
`933`	`933`	`asyncPayloadRecord,`
`934`	`934`	`);`
`935`	`935`	`if (isPromise(completed)) {`
`936`		`- // see: https://github.com/tc39/proposal-faster-promise-adoption`
`937`		`- // it is faster to await a promise prior to returning it from an async function`
`938`	`936`	`completed = await completed;`
`939`	`937`	`}`
`940`	`938`	`return completed;`