Skip to content

[s390x] Use consistent shift instruction#107559

Merged
lambdageek merged 1 commit intodotnet:mainfrom
saitama951:fix-slag
Sep 11, 2024
Merged

[s390x] Use consistent shift instruction#107559
lambdageek merged 1 commit intodotnet:mainfrom
saitama951:fix-slag

Conversation

@saitama951
Copy link
Copy Markdown
Contributor

@saitama951 saitama951 commented Sep 9, 2024

Fixes #107387
The slag instructions throws an undefined behavior when moving from Float to Int in OP_MOVE_I4_TO_F

@ghost ghost added the area-Codegen-JIT-mono label Sep 9, 2024
@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Sep 9, 2024
Copy link
Copy Markdown
Member

@lambdageek lambdageek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(caveat: I don't know s390x assembly - just reviewing for code structure)

@saitama951
Copy link
Copy Markdown
Contributor Author

Can we merge this?

@lambdageek lambdageek merged commit 4930e1b into dotnet:main Sep 11, 2024
@saitama951
Copy link
Copy Markdown
Contributor Author

@lambdageek can we backport this to release/9.0?

@lambdageek
Copy link
Copy Markdown
Member

/backport to release/9.0

@github-actions
Copy link
Copy Markdown
Contributor

Started backporting to release/9.0: https://github.com/dotnet/runtime/actions/runs/10836189535

jtschuster pushed a commit to jtschuster/runtime that referenced this pull request Sep 17, 2024
Fixes dotnet#107387
The slag instructions throws an undefined behavior when moving from Float to Int in OP_MOVE_I4_TO_F
sirntar pushed a commit to sirntar/runtime that referenced this pull request Sep 30, 2024
Fixes dotnet#107387
The slag instructions throws an undefined behavior when moving from Float to Int in OP_MOVE_I4_TO_F
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-s390x Related to s390x architecture (unsupported) area-Codegen-JIT-mono community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[s390x] Undefined behaviour in float32

2 participants