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

Ensure that the Create(Dot(...)) optimization doesn't kick in for Vector2 pre SSE4.1 #96951

Merged
merged 4 commits into from
Jan 17, 2024

Conversation

tannergooding
Copy link
Member

This resolves #96939

@ghost ghost assigned tannergooding Jan 14, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Jan 14, 2024
@ghost
Copy link

ghost commented Jan 14, 2024

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details

This resolves #96939

Author: tannergooding
Assignees: tannergooding
Labels:

area-CodeGen-coreclr

Milestone: -

@tannergooding
Copy link
Member Author

tannergooding commented Jan 16, 2024

CC. @dotnet/jit-contrib for review, this fixes #96939 and will need consideration for backporting to .NET 8

@EgorBo
Copy link
Member

EgorBo commented Jan 16, 2024

Are you planning to backport the fix to 8.0 ?

@tannergooding
Copy link
Member Author

Are you planning to backport the fix to 8.0 ?

Yes, but it will need to wait for the March release.

@tannergooding tannergooding merged commit cc59b89 into dotnet:main Jan 17, 2024
139 checks passed
@tannergooding tannergooding deleted the fix-96939 branch January 17, 2024 02:50
@tannergooding
Copy link
Member Author

/backport to release/8.0-staging

Copy link
Contributor

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/7550401659

tmds pushed a commit to tmds/runtime that referenced this pull request Jan 23, 2024
…tor2 pre SSE4.1 (dotnet#96951)

* Ensure that the Create(Dot(...)) optimization doesn't kick in for Vector2 pre SSE4.1

* Make sure to use #if defined(...)

* Add missing using

* Fix a type in the test
@github-actions github-actions bot locked and limited conversation to collaborators Feb 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tiered miscompilation of Vector2.Dot(x, x) without SSE4.1
2 participants