-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[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
[6.2] utils: Add llvm-symbolizer to install_components_with_clang presets #82007
Conversation
@swift-ci please smoke test |
@swift-ci please smoke test macOS platform |
@swift-ci please test macOS platform |
@swift-ci please test Linux platform |
@swift-ci please test macOS platform |
3 similar comments
@swift-ci please test macOS platform |
@swift-ci please test macOS platform |
@swift-ci please test macOS platform |
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? |
@swift-ci please test macOS platform |
@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.
Head branch was pushed to by a user without write access
4cd1d96
to
78f9dc6
Compare
@swift-ci please test macOS platform |
@swift-ci please test Linux platform |
@swift-ci please test macOS platform |
@swift-ci please smoke test |
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.
Extra llvm tool is added. Small increase in bundle size for swift.org toolchains.
Resolves swiftlang/swiftly#368
#81966
Very low. An extra llvm tool will be installed in macOS and Linux toolchain bundles
N/A
@shahmishal @FranzBusch