Skip to content

Fix Int128 checked-convert to signed IntX#100342

Merged
tannergooding merged 2 commits into
dotnet:mainfrom
skyoxZ:fix-int128-checked-convert
Apr 4, 2024
Merged

Fix Int128 checked-convert to signed IntX#100342
tannergooding merged 2 commits into
dotnet:mainfrom
skyoxZ:fix-int128-checked-convert

Conversation

@skyoxZ

@skyoxZ skyoxZ commented Mar 27, 2024

Copy link
Copy Markdown
Contributor

Fix #99489

Hope to add more tests for Int128 in the future.

@ghost ghost added the area-System.Numerics label Mar 27, 2024
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-system-numerics
See info in area-owners.md if you want to be subscribed.

@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Mar 27, 2024
Comment thread src/libraries/System.Private.CoreLib/src/System/Int128.cs Outdated
@skyoxZ

skyoxZ commented Mar 27, 2024

Copy link
Copy Markdown
Contributor Author

Feedback addressed.

@tannergooding
tannergooding merged commit 9a1da4d into dotnet:main Apr 4, 2024
@skyoxZ
skyoxZ deleted the fix-int128-checked-convert branch April 4, 2024 23:18
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
* Fix Int128 checked-convert to signed IntX

* Simplify implementation
@github-actions github-actions Bot locked and limited conversation to collaborators May 5, 2024
@tannergooding

Copy link
Copy Markdown
Member

/backport to release/8.0-staging

@github-actions github-actions Bot unlocked this conversation Aug 5, 2024
@github-actions

github-actions Bot commented Aug 5, 2024

Copy link
Copy Markdown
Contributor

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

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Numerics 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.

Overflows are ignored in some cases when checked-converting from Int128 to SignedInteger

2 participants