Skip to content

chore: bump instrument-hooks to d83209f for macos support#61

Draft
not-matthias wants to merge 5 commits into
mainfrom
cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos
Draft

chore: bump instrument-hooks to d83209f for macos support#61
not-matthias wants to merge 5 commits into
mainfrom
cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos

Conversation

@not-matthias
Copy link
Copy Markdown
Member

No description provided.

@not-matthias not-matthias force-pushed the cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos branch from 054a4d1 to a187704 Compare May 20, 2026 16:31
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 20, 2026

Merging this PR will improve performance by 13.89%

⚡ 2 improved benchmarks
✅ 46 untouched benchmarks
⏩ 1 skipped benchmark1

Performance Changes

Benchmark BASE HEAD Efficiency
bench_collect_results[5000000, 25] 3 s 2.7 s +12.42%
BenchmarkExternalFib 60 ns 52 ns +15.38%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos (e70400b) with main (4d99425)

Open in CodSpeed

Footnotes

  1. 1 benchmark was skipped, so the baseline result was used instead. If it was deleted from the codebase, click here and archive it to remove it from the performance reports.

Go 1.26+ invokes dsymutil on macOS to extract DWARF into a .dSYM bundle,
which can be OOM-killed on large cgo binaries. Walltime mode doesn't need
DWARF symbols, so disable them on macOS while keeping them on Linux for
simulation/callgrind profiling.
@not-matthias not-matthias force-pushed the cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos branch from d9a0c74 to f317ec1 Compare May 22, 2026 12:37
@not-matthias not-matthias force-pushed the cod-2639-bump-instrument-hooks-in-codspeed-go-to-support-macos branch from aab7bec to e70400b Compare May 22, 2026 12:46
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