Skip to content

[Linux] Make the backtracing symbol reference conditional. #68799

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

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

al45tair
Copy link
Contributor

On the LSan builds, we disable the backtracer, which results in a link error because we completely remove the code from libswiftCore.

rdar://116105222

On the LSan builds, we disable the backtracer, which results in a
link error because we completely remove the code from libswiftCore.

rdar://116105222
@al45tair al45tair requested a review from a team as a code owner September 27, 2023 09:03
@al45tair
Copy link
Contributor Author

@swift-ci Please test

@al45tair
Copy link
Contributor Author

This is a cherry pick of #68798.

@al45tair
Copy link
Contributor Author

al45tair commented Sep 27, 2023

Explanation: #68669 added a reference to a symbol in the backtracer's crash handling code. However, if the backtracer is disabled (as it is in LSan builds), that code is compiled out of the runtime, so the reference needed to be conditional.
Risk: Low. Only affects Linux.
Original PR: #68798
Reviewed by: @eeckstein
Resolves: rdar://116105222
Tests: The test from #68669 should still succeed after this change, and it shouldn't be running on the LSan build as backtracing is disabled there.

@al45tair al45tair added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels Sep 27, 2023
@al45tair al45tair merged commit 0a8d0e8 into swiftlang:release/5.9 Sep 28, 2023
al45tair added a commit to al45tair/swift that referenced this pull request Oct 2, 2023
[Linux] Make the backtracing symbol reference conditional.
al45tair added a commit to al45tair/swift that referenced this pull request Oct 2, 2023
[Linux] Make the backtracing symbol reference conditional.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.9
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants