-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[wasm] Switch default modules to es6 #70746
Conversation
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsMotivation, benefits
Challenges, downsides
Changes in this PR
|
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
3dfdc1a
to
5d07665
Compare
5d07665
to
e60425a
Compare
|
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
For the For the templates, I did just basics. I plan to do more work on it as part of #70892 |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
Co-authored-by: Ankit Jain <radical@gmail.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
The |
Motivation, benefits
Challenges, downsides
Changes in this PR
<WasmEnableES6>
default trueEXPORT_ES6=1
should not be in theemcc-link.rsp
because it could not be removed by workload re-link when<WasmEnableES6> == false
.