Skip to content

src: add support for top level await - #30370

Merged
devsnek merged 1 commit into
nodejs:masterfrom
devsnek:feature-tla
May 14, 2020
Merged

src: add support for top level await#30370
devsnek merged 1 commit into
nodejs:masterfrom
devsnek:feature-tla

Conversation

@devsnek

@devsnekdevsnek commented Nov 12, 2019•
edited
Loading

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Blockers

@devsnekdevsnek added vm Issues and PRs related to the vm subsystem. experimental Issues and PRs related to experimental features. esm Issues and PRs related to the ECMAScript Modules implementation. labels Nov 12, 2019
@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Nov 12, 2019
@devsnekdevsnek removed the c++ Issues and PRs that require attention from people who are familiar with C++. label Nov 12, 2019
@devsnek

devsnek commented Nov 12, 2019

Copy link
Copy Markdown
MemberAuthor

cc @nodejs/modules-active-members

TLA modules always evaluate to undefined, so this breaks node -pe for modules

devsnek
devsnek commented Nov 12, 2019
View reviewed changes
Comment threadsrc/module_wrap.cc Outdated
@devsnek

This comment has been minimized.

Sign in to view
@lin7sh

This comment has been minimized.

Sign in to view
@devsnek

This comment has been minimized.

Sign in to view
@devsnek
devsnekforce-pushed the feature-tla branch 3 times, most recently from 7479161 to 3bb22b2CompareNovember 12, 2019 09:07
@bnoordhuis

This comment has been minimized.

Sign in to view
@devsnek

This comment has been minimized.

Sign in to view
@bnoordhuis

This comment has been minimized.

Sign in to view
@devsnek

This comment has been minimized.

Sign in to view
@hybrist

This comment has been minimized.

Sign in to view
@devsnek

This comment has been minimized.

Sign in to view
@devsnek

devsnek commented Nov 12, 2019

Copy link
Copy Markdown
MemberAuthor

opened v8:9968 to address the cache issue.

@devsnekdevsnek added the blocked PRs that are blocked by other issues or PRs. label Nov 12, 2019
@guybedford

guybedford commented Nov 12, 2019

Copy link
Copy Markdown
Contributor

Great work. Are there any roadblocks to asyncifying node -pe lifecycle? Those are the only failing tests?

@devsnek

devsnek commented Nov 12, 2019•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

@guybedford it evaluates fine. the difference is that TLA modules always evaluate to undefined, so -p is kind of useless.

I'm not sure whether -p for modules should throw or just be undefined. Thoughts?

@guybedford

guybedford commented Nov 12, 2019

Copy link
Copy Markdown
Contributor

Surely the promise can still return the last expression output though? Perhaps a v8 feature?

@guybedford

guybedford commented Nov 12, 2019

Copy link
Copy Markdown
Contributor

I guess -p for modules makes no sense actually... Let's just make sure we properly throw for this case always.

@weswigham

This comment has been minimized.

Sign in to view
@devsnek

This comment has been minimized.

Sign in to view
@nodejs-github-bot

nodejs-github-bot commented May 12, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/31291/

@MylesBorins

MylesBorins commented May 12, 2020

Copy link
Copy Markdown
Contributor

If there are no objections I would like to land this in 48 hours

@devsnek

devsnek commented May 12, 2020

Copy link
Copy Markdown
MemberAuthor

@MylesBorins i'd like addaleax to review the changes i made to the options parser first.

frank-dspeed
frank-dspeed approved these changes May 13, 2020
View reviewed changes

@frank-dspeedfrank-dspeed left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment threadsrc/module_wrap.cc Outdated
@nodejs-github-bot

nodejs-github-bot commented May 13, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/31311/

@nodejs-github-bot

nodejs-github-bot commented May 14, 2020

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/31316/

@devsnek
src: add support for TLA
5ae5262
PR-URL: nodejs#30370
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@devsnek

devsnek commented May 14, 2020

Copy link
Copy Markdown
MemberAuthor

Landed in 5ae5262

@devsnek
devsnek merged commit 5ae5262 into nodejs:masterMay 14, 2020
@devsnek
devsnek deleted the feature-tla branch May 14, 2020 17:41
@devsnekdevsnek added the notable-change PRs with changes that should be highlighted in changelogs. label May 14, 2020
MylesBorins pushed a commit that referenced this pull request May 14, 2020
@devsnek@MylesBorins
src: add support for TLA
0d77eec
PR-URL: #30370
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
codebytere added a commit that referenced this pull request May 18, 2020
@codebytere
2020-05-19, Version 14.3.0 (Current)
2324d42
Notable changes:
async_hooks:
* (SEMVER-MINOR) move PromiseHook handler to JS (Stephen Belanger) #32891
cli:
* (SEMVER-MINOR) add `--trace-atomics-wait` flag (Anna Henningsen) #33292
fs:
* (SEMVER-MINOR) add .ref() and .unref() methods to watcher classes (rickyes) #33134
http:
* (SEMVER-MINOR) expose http.validate-header-name/value (osher) #33119
repl:
* (SEMVER-MINOR) deprecate repl._builtinLibs (Ruben Bridgewater) #33294
* (SEMVER-MINOR) remove obsolete completer variable (Ruben Bridgewater) #33294
* (SEMVER-MINOR) deprecate repl.inputStream and repl.outputStream (Ruben Bridgewater) #33294
* (SEMVER-MINOR) improve repl autocompletion for require calls (Ruben Bridgewater) #33282
* (SEMVER-MINOR) replace hard coded core module list with actual list (Ruben Bridgewater) #33282
* (SEMVER-MINOR) show reference errors during preview (Ruben Bridgewater) #33282
* (SEMVER-MINOR) improve repl preview (Ruben Bridgewater) #33282
src:
* add support for TLA (Gus Caplan) #30370
test:
* (SEMVER-MINOR) refactor test/parallel/test-bootstrap-modules.js (Ruben Bridgewater) #33282
PR-URL: TODO
@codebyterecodebytere mentioned this pull request May 18, 2020
Merged
codebytere added a commit that referenced this pull request May 19, 2020
@codebytere
2020-05-19, Version 14.3.0 (Current)
76a52dd
Notable changes:
async_hooks:
* (SEMVER-MINOR) move PromiseHook handler to JS (Stephen Belanger) #32891
cli:
* (SEMVER-MINOR) add `--trace-atomics-wait` flag (Anna Henningsen) #33292
fs:
* (SEMVER-MINOR) add .ref() and .unref() methods to watcher classes (rickyes) #33134
http:
* (SEMVER-MINOR) expose http.validate-header-name/value (osher) #33119
repl:
* (SEMVER-MINOR) deprecate repl._builtinLibs (Ruben Bridgewater) #33294
* (SEMVER-MINOR) deprecate repl.inputStream and repl.outputStream (Ruben Bridgewater) #33294
* (SEMVER-MINOR) show reference errors during preview (Ruben Bridgewater) #33282
* (SEMVER-MINOR) improve repl preview (Ruben Bridgewater) #33282
src:
* add support for TLA (Gus Caplan) #30370
PR-URL: TODO
codebytere added a commit that referenced this pull request May 19, 2020
@codebytere
2020-05-19, Version 14.3.0 (Current)
a33e87c
Notable changes:
async_hooks:
* (SEMVER-MINOR) move PromiseHook handler to JS (Stephen Belanger) #32891
cli:
* (SEMVER-MINOR) add `--trace-atomics-wait` flag (Anna Henningsen) #33292
fs:
* (SEMVER-MINOR) add .ref() and .unref() methods to watcher classes (rickyes) #33134
http:
* (SEMVER-MINOR) expose http.validate-header-name/value (osher) #33119
repl:
* (SEMVER-MINOR) deprecate repl._builtinLibs (Ruben Bridgewater) #33294
* (SEMVER-MINOR) deprecate repl.inputStream and repl.outputStream (Ruben Bridgewater) #33294
* (SEMVER-MINOR) show reference errors during preview (Ruben Bridgewater) #33282
* (SEMVER-MINOR) improve repl preview (Ruben Bridgewater) #33282
src:
* add support for TLA (Gus Caplan) #30370
PR-URL: TODO
codebytere added a commit that referenced this pull request May 19, 2020
@codebytere
2020-05-19, Version 14.3.0 (Current)
5d81e4d
Notable changes:
async_hooks:
* (SEMVER-MINOR) move PromiseHook handler to JS (Stephen Belanger) #32891
cli:
* (SEMVER-MINOR) add `--trace-atomics-wait` flag (Anna Henningsen) #33292
fs:
* (SEMVER-MINOR) add .ref() and .unref() methods to watcher classes (rickyes) #33134
http:
* (SEMVER-MINOR) expose http.validate-header-name/value (osher) #33119
repl:
* (SEMVER-MINOR) deprecate repl._builtinLibs (Ruben Bridgewater) #33294
* (SEMVER-MINOR) deprecate repl.inputStream and repl.outputStream (Ruben Bridgewater) #33294
* (SEMVER-MINOR) show reference errors during preview (Ruben Bridgewater) #33282
* (SEMVER-MINOR) improve repl preview (Ruben Bridgewater) #33282
src:
* add support for TLA (Gus Caplan) #30370
PR-URL: TODO
codebytere added a commit that referenced this pull request May 19, 2020
@codebytere
2020-05-19, Version 14.3.0 (Current)
c841f51
Notable changes:
async_hooks:
* (SEMVER-MINOR) move PromiseHook handler to JS (Stephen Belanger) #32891
cli:
* (SEMVER-MINOR) add `--trace-atomics-wait` flag (Anna Henningsen) #33292
fs:
* (SEMVER-MINOR) add .ref() and .unref() methods to watcher classes (rickyes) #33134
http:
* (SEMVER-MINOR) expose http.validate-header-name/value (osher) #33119
repl:
* (SEMVER-MINOR) deprecate repl._builtinLibs (Ruben Bridgewater) #33294
* (SEMVER-MINOR) deprecate repl.inputStream and repl.outputStream (Ruben Bridgewater) #33294
* (SEMVER-MINOR) show reference errors during preview (Ruben Bridgewater) #33282
* (SEMVER-MINOR) improve repl preview (Ruben Bridgewater) #33282
src:
* add support for TLA (Gus Caplan) #30370
PR-URL: #33452
@codebytere

codebytere commented Jun 9, 2020

Copy link
Copy Markdown
Member

@devsnek can this go back to v12.x? I'm not sure if the version of v8 on that branch is compatible, so i figured i'd ask before trying

@devsnek

devsnek commented Jun 9, 2020

Copy link
Copy Markdown
MemberAuthor

@codebytere i don't think anything here is compatible.

@frank-dspeedfrank-dspeed mentioned this pull request Jul 24, 2020
Closed
@aduh95aduh95 mentioned this pull request Oct 30, 2021
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GeoffreyBoothGeoffreyBoothGeoffreyBooth left review comments
@addaleaxaddaleaxaddaleax left review comments
@benjamingrbenjamingrbenjamingr approved these changes
@targostargostargos left review comments
@guybedfordguybedfordguybedford approved these changes
+2 more reviewers
@MylesBorinsMylesBorinsMylesBorins approved these changes
@frank-dspeedfrank-dspeedfrank-dspeed approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

esmIssues and PRs related to the ECMAScript Modules implementation.experimentalIssues and PRs related to experimental features.notable-changePRs with changes that should be highlighted in changelogs.vmIssues and PRs related to the vm subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

18 participants

@devsnek@lin7sh@bnoordhuis@hybrist@guybedford@weswigham@littledan@addaleax@BridgeAR@nodejs-github-bot@lundibundi@frank-dspeed@tomasgvivo@MylesBorins@GeoffreyBooth@benjamingr@codebytere@targos