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
For perf, use the new ConstructorInvoker APIs for ActivatorUtilities.CreateFactory #90119
Conversation
Tagging subscribers to this area: @dotnet/area-extensions-dependencyinjection Issue DetailsRe-commit the original + a fix for NativeAot.
|
src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/ConstructorInvoker.cs
Show resolved
Hide resolved
src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/ConstructorInvoker.cs
Outdated
Show resolved
Hide resolved
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/ConstructorInvoker.cs
Outdated
Show resolved
Hide resolved
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the CI (including NAOT runtime-extra-platforms legs) is green.
This should fix #90039 |
The runtime-extra-platforms CI errors appear unrelated; created some new tracking issues. Verified library tests with NativeAot locally. |
Re-commit the original + fix NativeAot + make core API validation consistent with NativeAot.