Skip to content

[6.2] utils: Add llvm-symbolizer to install_components_with_clang presets #82007

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ADKaster
Copy link
Contributor

@ADKaster ADKaster commented Jun 5, 2025

  • Explanation:

llvm-symbolizer is critical to ASAN/LSAN properly reading and applying allowlists and suppressions. Without properly symbolized backtraces, the tools cannot match stack traces to suppression lists, leading to false negatives in leak detection and other issues.

Ensure that it's installed in the toolchain presets.

  • Scope:

Extra llvm tool is added. Small increase in bundle size for swift.org toolchains.

  • Issues:

Resolves swiftlang/swiftly#368

  • Original PRs:

#81966

  • Risk:

Very low. An extra llvm tool will be installed in macOS and Linux toolchain bundles

  • Testing:

N/A

  • Reviewers:

@shahmishal @FranzBusch

@ADKaster ADKaster requested a review from a team as a code owner June 5, 2025 02:22
@ADKaster
Copy link
Contributor Author

ADKaster commented Jun 5, 2025

As noted by @compnerd in #81966, Windows toolchains already bundle this tool.

@FranzBusch
Copy link
Member

@swift-ci please smoke test

@FranzBusch
Copy link
Member

@swift-ci please smoke test macOS platform

@FranzBusch FranzBusch enabled auto-merge June 5, 2025 14:38
@compnerd
Copy link
Member

compnerd commented Jun 6, 2025

@swift-ci please test macOS platform

@compnerd
Copy link
Member

compnerd commented Jun 6, 2025

@swift-ci please test Linux platform

@FranzBusch
Copy link
Member

@swift-ci please test macOS platform

3 similar comments
@compnerd
Copy link
Member

compnerd commented Jun 7, 2025

@swift-ci please test macOS platform

@compnerd
Copy link
Member

compnerd commented Jun 7, 2025

@swift-ci please test macOS platform

@FranzBusch
Copy link
Member

@swift-ci please test macOS platform

@ADKaster
Copy link
Contributor Author

Is there something to do to this branch to get the CI happy enough to back port? Is adding this extra tool breaking some build time constraint?

@FranzBusch
Copy link
Member

@swift-ci please test macOS platform

@FranzBusch
Copy link
Member

@ADKaster can you rebase against the 6.2 branch just to see if that fixes it?

llvm-symbolizer is critical to ASAN/LSAN properly reading and applying
allowlists and suppressions. Without properly symbolized backtraces,
the tools cannot match stack traces to suppression lists, leading to
false negatives in leak detection and other issues.

Ensure that it's installed in the toolchain presets.
auto-merge was automatically disabled June 16, 2025 16:19

Head branch was pushed to by a user without write access

@ADKaster ADKaster force-pushed the backport-llvm-symbolizer-base-toolchain branch from 4cd1d96 to 78f9dc6 Compare June 16, 2025 16:19
@FranzBusch
Copy link
Member

@swift-ci please test macOS platform

@FranzBusch
Copy link
Member

@swift-ci please test Linux platform

@FranzBusch
Copy link
Member

@swift-ci please test macOS platform

@FranzBusch
Copy link
Member

@swift-ci please smoke test

@FranzBusch FranzBusch enabled auto-merge June 17, 2025 23:28
@FranzBusch FranzBusch merged commit 5384c54 into swiftlang:release/6.2 Jun 18, 2025
4 of 5 checks passed
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