Skip to content

Fix benchmark runner formatting check - #7

Closed
wlmrh wants to merge 1 commit into
mainfrom
fix/benchmark-runner-format
Closed

Fix benchmark runner formatting check#7
wlmrh wants to merge 1 commit into
mainfrom
fix/benchmark-runner-format

Conversation

@wlmrh

@wlmrh wlmrh commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • format benchmark/llama_engine_benchmark.cpp with clang-format 14.0.6
  • address the formatting failure observed in PR Realistic benchmark reporting #6 and the merged main CI run
  • make no runtime, benchmark behavior, or report-data changes

Verification

  • full repository clang-format-14 --dry-run --Werror check passes
  • Python syntax check passes
  • 21 benchmark utility tests pass
  • CPU warnings-as-errors build completes locally

Local platform note

The macOS CTest run completed 59 tests successfully, skipped 7 model-dependent tests, and retained one pre-existing platform-sensitive RoPE tolerance failure (1.24e-5 difference against a 1e-5 tolerance). This PR does not modify that test or runtime behavior.

@wlmrh wlmrh closed this Jul 21, 2026
@wlmrh
wlmrh deleted the fix/benchmark-runner-format branch August 5, 2026 08:38
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.

1 participant