Skip to content

Commit

Permalink
don't use target-cpu=native for the benchmarks
Browse files Browse the repository at this point in the history
It's an unfair advantage versus zlib-ng, and any changes we make to dynamically recognizing features would not get picked up by the benchmarks
  • Loading branch information
folkertdev committed Jan 8, 2025
1 parent 881c049 commit acd2c0a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/bench.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
- name: cargo build
run: |
. "$HOME/.cargo/env"
RUSTFLAGS="-Ctarget-cpu=native -Cllvm-args=-enable-dfa-jump-thread" cargo build --target ${{matrix.target}} -p test-libz-rs-sys --release --examples
RUSTFLAGS="-Cllvm-args=-enable-dfa-jump-thread" cargo build --target ${{matrix.target}} -p test-libz-rs-sys --release --examples
cp target/${{matrix.target}}/release/examples/blogpost-compress .
cp target/${{matrix.target}}/release/examples/blogpost-uncompress .
- name: Benchmark
Expand Down

0 comments on commit acd2c0a

Please sign in to comment.