Skip to content

Add AggressiveInlining attribute to multi-dimension array helpers - #60750

Closed
BruceForstall wants to merge 1 commit into
dotnet:mainfrom
BruceForstall:AggressiveInlineMultiDimHelpers
Closed

BruceForstall wants to merge 1 commit into
dotnet:mainfrom
BruceForstall:AggressiveInlineMultiDimHelpers

Conversation

@BruceForstall

Copy link
Copy Markdown
Contributor

Specifically,

Rank.get
GetLength
GetUpperBound
GetLowerBound

These can cause significant code growth, however, they are important
to inline for the performance of multi-dimensional array loops that use them.

Specifically,

Rank.get
GetLength
GetUpperBound
GetLowerBound

These can cause significant code growth, however, they are important
to inline for the performance of many multi-dimensional array loops.
@ghost

ghost commented Oct 22, 2021

Copy link
Copy Markdown

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@BruceForstall

Copy link
Copy Markdown
Contributor Author

When measured with the test in dotnet/performance#2093, a 50% execution time improvement is seen with this change.

@ghost ghost locked as resolved and limited conversation to collaborators Nov 24, 2021
@BruceForstall
BruceForstall deleted the AggressiveInlineMultiDimHelpers branch December 28, 2022 01:15
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.

1 participant