Skip to content

Add flags support to benchmarks to report log/set/userspace benchmarks specificities#546

Merged
qdeslandes merged 2 commits into
facebook:mainfrom
qdeslandes:report_benchmark_flags
May 19, 2026
Merged

Add flags support to benchmarks to report log/set/userspace benchmarks specificities#546
qdeslandes merged 2 commits into
facebook:mainfrom
qdeslandes:report_benchmark_flags

Conversation

@qdeslandes
Copy link
Copy Markdown
Contributor

Benchmarks results can be influenced by the matchers used:

  • Logging benchmarks are more volatile, so are benchmarks using sets
  • Some benchmarks do not generate bytecode, but bench the generation step

Those flags are reflected in the benchmark results, and in the HTML report.

Those flags will be used to later in bfoptimizer to detect benchmarks results to ignore.

@meta-cla meta-cla Bot added the cla signed label May 19, 2026
@qdeslandes qdeslandes enabled auto-merge (rebase) May 19, 2026 01:53
@qdeslandes qdeslandes disabled auto-merge May 19, 2026 02:00
@qdeslandes qdeslandes force-pushed the report_benchmark_flags branch from b0a2fad to 2928bd3 Compare May 19, 2026 02:08
Introduce bft::benchmark::Flags and set_flags() to annotate benchmarks
with use_set, use_log, and userspace_only counters. These appear in the
JSON output so tooling can filter out flaky or non-execution benchmarks
without relying on name prefixes.
Parse use_set, use_log, and userspace_only counters from the benchmark
JSON output into Result, bubble them up through Benchmark and
BenchmarkRow, and render them as badge pills in the HTML report table.
@qdeslandes qdeslandes force-pushed the report_benchmark_flags branch from 2928bd3 to aedfd27 Compare May 19, 2026 02:41
@qdeslandes qdeslandes enabled auto-merge (rebase) May 19, 2026 02:41
@qdeslandes qdeslandes merged commit 9036e1b into facebook:main May 19, 2026
38 checks passed
@qdeslandes qdeslandes deleted the report_benchmark_flags branch May 19, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant