Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[benchmark] add gbenchmark #2780

Merged
merged 25 commits into from
Jul 1, 2022
Merged

Conversation

strega-nil-ms
Copy link
Contributor

@strega-nil-ms strega-nil-ms commented Jun 10, 2022

This adds the google benchmark library, along with one benchmark from @BillyONeal's test benchmarking branch.

One "enjoyable" thing about having a benchmarking library with pre-built binaries is that we have to make sure there isn't any conflict with the STL; this means that we need to make sure that the library links with our STL.

Given that we build multiple configurations in one build, which is Not Normal CMake, this means that we need to choose one configuration for the benchmarking library to link to - we do this with cache variables. We default to dynamic release mode.

To Do

@CaseyCarter CaseyCarter added the infrastructure Related to repository automation label Jun 10, 2022
NOTICE.txt Outdated Show resolved Hide resolved
docs/cgmanifest.json Outdated Show resolved Hide resolved
.gitmodules Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/inc/udt.hpp Outdated Show resolved Hide resolved
benchmarks/inc/udt.hpp Outdated Show resolved Hide resolved
.gitmodules Outdated Show resolved Hide resolved
stl/CMakeLists.txt Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
.gitmodules Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
stl/CMakeLists.txt Show resolved Hide resolved
stl/CMakeLists.txt Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
@strega-nil-ms strega-nil-ms marked this pull request as ready for review June 14, 2022 19:35
@strega-nil-ms strega-nil-ms requested a review from a team as a code owner June 14, 2022 19:35
@strega-nil-ms

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
stl/CMakeLists.txt Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
.gitmodules Outdated Show resolved Hide resolved
benchmarks/inc/udt.hpp Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
benchmarks/inc/udt.hpp Outdated Show resolved Hide resolved
benchmarks/src/copy.cpp Outdated Show resolved Hide resolved
benchmarks/src/copy.cpp Outdated Show resolved Hide resolved
benchmarks/src/copy.cpp Outdated Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
stl/CMakeLists.txt Outdated Show resolved Hide resolved
@azure-pipelines

This comment was marked as resolved.

CMakeLists.txt Show resolved Hide resolved
@CaseyCarter CaseyCarter self-requested a review June 23, 2022 06:24
@StephanTLavavej

This comment was marked as resolved.

@strega-nil-ms

This comment was marked as resolved.

NOTICE.txt Outdated Show resolved Hide resolved
Copy link
Member

@StephanTLavavej StephanTLavavej left a comment

Choose a reason for hiding this comment

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

Thanks! Just a few minor issues, I will double-check and push changes.

azure-devops/checkout-sources.yml Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
benchmarks/inc/utility.hpp Show resolved Hide resolved
benchmarks/inc/xoshiro.hpp Show resolved Hide resolved
benchmarks/CMakeLists.txt Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@StephanTLavavej StephanTLavavej removed their assignment Jun 30, 2022
@StephanTLavavej
Copy link
Member

@strega-nil-ms @CaseyCarter I pushed a small set of changes - if anything looks wrong, please meow! Otherwise I believe this is ready. Thanks for this major step forward! 😸

@StephanTLavavej StephanTLavavej self-assigned this Jun 30, 2022
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 088818b into microsoft:main Jul 1, 2022
@StephanTLavavej
Copy link
Member

Thanks for implementing this long-awaited infrastructure, and congratulations for this major PR! 🏎️ 🚀 😻

@strega-nil strega-nil deleted the benchmarkin branch August 10, 2022 20:35
fsb4000 pushed a commit to fsb4000/STL that referenced this pull request Aug 13, 2022
Co-authored-by: Nicole Mazzuca <[email protected]>
Co-authored-by: Stephan T. Lavavej <[email protected]>
Co-authored-by: Casey Carter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Related to repository automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants