Skip to content

Provide helpers to create integer lists for the given ranges. #1179

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 16, 2021
Merged

Provide helpers to create integer lists for the given ranges. #1179

merged 1 commit into from
Jun 16, 2021
+48 −0

Conversation

PCMan
Copy link
Contributor

@PCMan PCMan commented Jun 16, 2021 •
edited
Loading

This can be used together with ArgsProduct() to allow multiple ranges
with different multipliers and mixing dense and sparse ranges.

Example:

BENCHMARK(MyTest)->ArgsProduct({
  CreateRange(0, 1024, /*multi=*/32),
  CreateRange(0, 100, /*multi=*/4),
  CreateDenseRange(0, 4, /*step=*/1)
});

@google-cla google-cla bot added the cla: yes label Jun 16, 2021
@PCMan PCMan force-pushed the add_range_helpers branch from 0a2310f to dfda085 Compare June 16, 2021 09:37
dmah42
dmah42 reviewed Jun 16, 2021
View reviewed changes
@PCMan PCMan force-pushed the add_range_helpers branch from dfda085 to de04d36 Compare June 16, 2021 09:42
dmah42
dmah42 reviewed Jun 16, 2021
View reviewed changes
dmah42
dmah42 reviewed Jun 16, 2021
View reviewed changes
Provide helpers to create integer lists for the given ranges.
8ee3b33 
This can be used together with ArgsProduct() to allow multiple ranges
with different multipliers and mixing dense and sparse ranges.

Example:
BENCHMARK(MyTest)->ArgsProduct({
  CreateRange(0, 1024, /*multi=*/32),
  CreateRange(0, 100, /*multi=*/4),
  CreateDenseRange(0, 4, /*step=*/1)
});
@PCMan PCMan force-pushed the add_range_helpers branch from de04d36 to 8ee3b33 Compare June 16, 2021 10:35
@dmah42 dmah42 merged commit c932169 into google:main Jun 16, 2021
@dmah42
Copy link
Member

dmah42 commented Jun 16, 2021

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmah42 dmah42 dmah42 left review comments

Assignees

No one assigned

Labels

cla: yes enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@PCMan @dmah42