Skip to content

Conversation

@lilinus
Copy link
Contributor

@lilinus lilinus commented Jan 28, 2024

Use Math.DivRem(uint, uint) and Math.DivRem(ulong, ulong) in decimal.

There are also many 64 by 32 (32-bit remainder with some using 64-bit quotient and some truncating to 64-bit quotient). Wasn't sure if we want to put those in some internal method, thoughts?

@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.Numerics labels Jan 28, 2024
@ghost
Copy link

ghost commented Jan 28, 2024

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

Issue Details

Use Math.DivRem(uint, uint) and Math.DivRem(ulong, ulong) in decimal.

There are also many 64 by 32 (32-bit remainder with some using 64-bit quotient and some truncating to 64-bit quotient). Wasn't sure if we want to put those in some internal method, thoughts?

Author: lilinus
Assignees: -
Labels:

area-System.Numerics, community-contribution

Milestone: -

@huoyaoyuan
Copy link
Member

I remember some ones can be better simplified with X86Base.DivRem, which is still not implemented in Mono yet.

@huoyaoyuan
Copy link
Member

@tannergooding tannergooding merged commit c794d03 into dotnet:main Jan 30, 2024
@lilinus lilinus deleted the decimal.deccalc-divrem branch February 17, 2024 13:16
@github-actions github-actions bot locked and limited conversation to collaborators Mar 19, 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.

3 participants