Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Conversation

@benaadams
Copy link
Member

@AndyAyersMS
Copy link
Member

Seems reasonable... in my experiments it looks like it reduces code size, both for value and ref type Ts (the latter when all is inlined into a non-shared method so T is known -- for shared root methods most inlines are blocked anyways because of shared/shared limitations).

@jkotas
Copy link
Member

jkotas commented May 29, 2018

@dotnet-bot test this please

@jkotas jkotas merged commit 86c438d into dotnet:master May 30, 2018
dotnet-bot pushed a commit to dotnet/corert that referenced this pull request May 30, 2018
jkotas pushed a commit to dotnet/corert that referenced this pull request May 31, 2018
dotnet-bot pushed a commit to dotnet/corefx that referenced this pull request Jun 10, 2018
Signed-off-by: dotnet-bot-corefx-mirror <[email protected]>
Anipik pushed a commit to dotnet/corefx that referenced this pull request Jun 10, 2018
Signed-off-by: dotnet-bot-corefx-mirror <[email protected]>
@benaadams benaadams deleted the inline-Span-copyTo branch August 20, 2018 18:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants