Skip to content

Conversation

@fhanau
Copy link
Contributor

@fhanau fhanau commented Apr 5, 2025

[build] 04-05-25 Build maintenance

  • Build simdutf using SSE4.2 to avoid having to include fallback code path, reducing code size. We already require SSE4.2 for zlib, so this won't break compatibility with any systems.
  • Minor Bazel/include cleanup based on IWYU
  • Clean up redundant Bazel arguments
  • Clean up comments based on no longer using Dawn

[build] Build using lld 20 on macOS
lld20 introduces a symbol string deduplication pass, reducing binary sizes on macOS (llvm/llvm-project#123874). For debug binaries with linker optimizations turned on, binary size improves by 930KB to 235.4MB, gains for release binaries should be similar.
Run brew update on CI to make this version available.

- Build simdutf using SSE4.2 to avoid having to include fallback code path, reducing code size. We already require SSE4.2 for zlib, so this won't break compatibility with any systems.
- Minor Bazel/include cleanup based on IWYU
- Clean up redundant Bazel arguments
- Clean up comments based on no longer using Dawn
@fhanau fhanau force-pushed the felix/040525-build-cleanup branch from 4800058 to 94c0254 Compare April 6, 2025 19:03
lld20 introduces a symbol string deduplication pass, reducing binary sizes on macOS (llvm/llvm-project#123874). For debug binaries with linker optimizations turned on, binary size improves by 930KB to 235.4MB, gains for release binaries should be similar.
Run `brew update` on CI to make this version available.
@fhanau fhanau marked this pull request as ready for review April 6, 2025 20:48
@fhanau fhanau requested review from a team as code owners April 6, 2025 20:48
@fhanau fhanau requested review from erikcorry and mar-cf April 6, 2025 20:48
@fhanau fhanau merged commit 385b77b into main Apr 7, 2025
17 checks passed
@fhanau fhanau deleted the felix/040525-build-cleanup branch April 7, 2025 15:08
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.

4 participants