Skip to content

[Backtracing] Make StackOverflow test less sensitive. #66433

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
Jun 11, 2023

Conversation

al45tair
Copy link
Contributor

@al45tair al45tair commented Jun 7, 2023

Depending on the platform, architecture and stack layout, we may or may not have line numbers for the very first frame.

rdar://110418951

Depending on the platform, architecture and stack layout, we may or
may not have line numbers for the very first frame.

rdar://110418951
@al45tair al45tair added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels Jun 7, 2023
@al45tair al45tair requested a review from a team as a code owner June 7, 2023 21:01
@al45tair
Copy link
Contributor Author

al45tair commented Jun 7, 2023

@swift-ci Please test

@al45tair
Copy link
Contributor Author

al45tair commented Jun 8, 2023

Explanation: The StackOverflow backtracing test needs to be more relaxed about the first line in the backtrace. Depending on exactly where the PC is when the stack actually overflows (which is architecture and platform specific), we may be in a place where we don't have line numbers (e.g. the function prologue). This was causing the test to fail on UBI9-aarch64 builds.
Risk: Low. Makes the test more relaxed; only affects a test.
Original PR: #66432
Reviewed by: @mikeash
Resolves: rdar://110418951
Tests: This fixes a test to be more forgiving.

@tshortli
Copy link
Contributor

Given that this is a test-only change, merging without branch manager approval to fix the bots.

@tshortli tshortli merged commit dab4e94 into swiftlang:release/5.9 Jun 11, 2023
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