-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[release/7.0-rc1] Disable matching constants for vectors that needs upper half to be save/restore #74192
[release/7.0-rc1] Disable matching constants for vectors that needs upper half to be save/restore #74192
Conversation
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsBackport of #74110 to release/7.0-rc1 /cc @kunalspathak Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
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.
approved. Please get a code review, and once we have a green ci we can merge.
@dotnet/jit-contrib @BruceForstall |
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.
approved. once we get a clean ci we can merge.
Approved and signed off. |
Backport of #74110 to release/7.0-rc1
/cc @kunalspathak
Customer Impact
There was a silent bad codegen issue that would result in wrong result in some cases where
Vector256
are involved.Testing
Existing test.
Risk
Compared to .NET 6, nothing. #70171 did see few improvements that we won't see in certain benchmarks.
IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.