Skip to content

Conversation

@BruceForstall
Copy link
Contributor

This is similar to MDArray2, but uses the GetLowerBound/GetUpperBound
APIs.

The iteration count is lowered compared to MDArray2 because using these
APIs slows the benchmark considerably.

This is similar to MDArray2, but uses the GetLowerBound/GetUpperBound
APIs.

The iteration count is lowered compared to MDArray2 because using these
APIs slows the benchmark considerably.
@BruceForstall
Copy link
Contributor Author

The test failures are not in the microbenchmarks.

@BruceForstall
Copy link
Contributor Author

@DrewScoggins PTAL

@BruceForstall
Copy link
Contributor Author

@DrewScoggins @adamsitnik @DrewScoggins Who should review and merge runtime micro-benchmark additions/changes? E.g., the JIT team doesn't have commit permission to this repo.

@DrewScoggins
Copy link
Member

Sorry, I was looking at this a couple days ago and got distracted, and never got back around to this.

The change LGTM, I will just merge it.

@DrewScoggins DrewScoggins merged commit bd17aa9 into dotnet:main Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants