Skip to content

[mono][aot] Avoid emitting gshared methods multiple times when prefer… #80100

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

Merged
merged 1 commit into from
Jan 5, 2023

Conversation

vargaz
Copy link
Contributor

@vargaz vargaz commented Jan 2, 2023

…ring instances of them.

This fixes a small amount of the size regression added by c7c6aa0.

@ghost ghost added the area-Codegen-AOT-mono label Jan 2, 2023
@ghost ghost assigned vargaz Jan 2, 2023
@vargaz vargaz force-pushed the prefer-instances2 branch from d331129 to 2a5d23d Compare January 2, 2023 17:11
@vargaz
Copy link
Contributor Author

vargaz commented Jan 2, 2023

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@build-analysis build-analysis bot mentioned this pull request Jan 2, 2023
@SamMonoRT
Copy link
Member

cc @kotlarmilos @ivanpovazan

…ring instances of them.

This fixes a part of the size regression added by c7c6aa0.
@vargaz vargaz force-pushed the prefer-instances2 branch from 2a5d23d to c111208 Compare January 4, 2023 15:11
@vargaz
Copy link
Contributor Author

vargaz commented Jan 4, 2023

/azp run runtime-wasm-libtests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vargaz vargaz merged commit 5c36a27 into dotnet:main Jan 5, 2023
@vargaz vargaz deleted the prefer-instances2 branch January 5, 2023 21:29
@ghost ghost locked as resolved and limited conversation to collaborators Feb 5, 2023
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.

3 participants