Skip to content

Optimize integral ToString (without generics)#32528

Merged
jkotas merged 1 commit intodotnet:masterfrom
ts2do:IntFormatNoGeneric
Feb 19, 2020
Merged

Optimize integral ToString (without generics)#32528
jkotas merged 1 commit intodotnet:masterfrom
ts2do:IntFormatNoGeneric

Conversation

@ts2do
Copy link
Contributor

@ts2do ts2do commented Feb 19, 2020

  • Make the fast path for Number.FormatXX & Number.TryFormatXX inlineable
  • Make parameterless ToString in SByte, Int16, Int32, and Int64 directly invoke the default formatting method

This includes the changes from #32413 without the use of generics.
@jkotas: Can #32413 be used as the 'other' PR or should I remove the optimizations and open another?

- Make the fast path for Number.FormatXX & Number.TryFormatXX inlineable
- Make parameterless ToString in SByte, Int16, Int32, and Int64 directly invoke the default formatting method
@jkotas
Copy link
Member

jkotas commented Feb 19, 2020

@jkotas: Can #32413 be used as the 'other' PR

Yes, that's fine.

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thank you!

@jkotas jkotas added tenet-performance Performance related issue area-System.Runtime labels Feb 19, 2020
@jkotas jkotas merged commit 7c7e8ed into dotnet:master Feb 19, 2020
@ts2do ts2do deleted the IntFormatNoGeneric branch February 20, 2020 03:21
@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants