Skip to content

test changing opt-level to z#3238

Open
codeshaunted wants to merge 1 commit intocanaryfrom
avery/opt-level-z
Open

test changing opt-level to z#3238
codeshaunted wants to merge 1 commit intocanaryfrom
avery/opt-level-z

Conversation

@codeshaunted
Copy link
Collaborator

@codeshaunted codeshaunted commented Mar 12, 2026

Summary by CodeRabbit

  • Chores
    • Optimized release build configuration for improved runtime performance.

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
beps Ready Ready Preview, Comment Mar 12, 2026 5:55pm
promptfiddle Ready Ready Preview, Comment Mar 12, 2026 5:55pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0b2ae02d-631d-4f48-b9c2-f08bb67e464e

📥 Commits

Reviewing files that changed from the base of the PR and between bd85df2 and 11284e9.

📒 Files selected for processing (1)
  • baml_language/Cargo.toml

📝 Walkthrough

Walkthrough

Configuration change to the Rust release profile in the Cargo.toml manifest, updating the optimization level from "s" (size-optimized) to "z" (maximum size reduction). No functional or behavioral modifications.

Changes

Cohort / File(s) Summary
Build Configuration
baml_language/Cargo.toml
Updated release profile opt-level from "s" to "z" for more aggressive binary size optimization.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'test changing opt-level to z' accurately describes the main change: updating the Rust release profile's opt-level from 's' to 'z' in Cargo.toml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch avery/opt-level-z
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 12, 2026

Merging this PR will degrade performance by 13.64%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

❌ 4 regressed benchmarks
✅ 4 untouched benchmarks
🆕 7 new benchmarks
⏩ 98 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
🆕 WallTime bench_incremental_add_user_field N/A 1.7 ms N/A
WallTime bench_single_simple_file 1.1 ms 1.2 ms -13.35%
🆕 WallTime bench_incremental_close_string N/A 1.3 ms N/A
WallTime bench_incremental_add_new_file 269 µs 311.5 µs -13.64%
WallTime bench_empty_project 928.3 µs 1,072.4 µs -13.44%
WallTime bench_incremental_add_field 263.4 µs 302.4 µs -12.9%
🆕 WallTime bench_incremental_add_attribute N/A 1.3 ms N/A
🆕 WallTime bench_incremental_rename_type N/A 2.1 ms N/A
🆕 WallTime bench_scale_deep_nesting N/A 4 ms N/A
🆕 WallTime bench_incremental_add_string_char N/A 1.3 ms N/A
🆕 WallTime bench_scale_100_functions N/A 2.5 ms N/A

Comparing avery/opt-level-z (11284e9) with canary (bd85df2)

Open in CodSpeed

Footnotes

  1. 98 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@github-actions
Copy link

Binary size checks passed

7 passed

Artifact Platform Gzip Baseline Delta Status
bridge_cffi Linux 4.3 MB 4.5 MB -201.6 KB (-4.5%) OK
bridge_cffi-stripped Linux 2.8 MB 3.0 MB -190.1 KB (-6.3%) OK
bridge_cffi macOS 3.5 MB 3.6 MB -69.7 KB (-1.9%) OK
bridge_cffi-stripped macOS 2.3 MB 2.3 MB -51.6 KB (-2.2%) OK
bridge_cffi Windows 3.3 MB 3.6 MB -291.1 KB (-8.2%) OK
bridge_cffi-stripped Windows 2.1 MB 2.4 MB -247.3 KB (-10.4%) OK
bridge_wasm WASM 2.0 MB 2.2 MB -135.8 KB (-6.2%) OK

Generated by cargo size-gate · workflow run

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