Skip to content
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] optimize interop startup and linker hints #77256

Merged
merged 1 commit into from
Dec 9, 2022

Conversation

pavelsavara
Copy link
Member

@pavelsavara pavelsavara commented Oct 20, 2022

Prepare for making legacy interop optional

  • move methods used only in legacy into LegacyHostImplementation class
  • rename GetTaskResult to GetTaskResultDynamic
  • make s_taskGetResultMethodInfo initialization lazy
  • rationalize DynamicDependency

@ghost
Copy link

ghost commented Oct 20, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

new FF AppStart, Reach managed: 355.1764705882353ms
new chrome AppStart, Reach managed: 249.43478260869566ms
old FF AppStart, Reach managed: 359.3529411764706ms
old chrome AppStart, Reach managed: 256.6ms

Author: pavelsavara
Assignees: -
Labels:

arch-wasm, area-System.Runtime.InteropServices.JavaScript

Milestone: 8.0.0

@pavelsavara pavelsavara changed the title [wasm] optimize interop startup [wasm] optimize interop startup and linker hints Oct 27, 2022
@ghost ghost closed this Nov 26, 2022
@ghost
Copy link

ghost commented Nov 26, 2022

Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it.

@pavelsavara
Copy link
Member Author

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@pavelsavara pavelsavara requested a review from kg December 8, 2022 14:35
@pavelsavara pavelsavara marked this pull request as ready for review December 8, 2022 18:12
@pavelsavara
Copy link
Member Author

CI failure is #70985

@pavelsavara pavelsavara merged commit add75d0 into dotnet:main Dec 9, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jan 8, 2023
@pavelsavara pavelsavara deleted the wasm_optimize_interop_startup branch September 2, 2024 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants