Skip to content

Conversation

@eeckstein
Copy link
Contributor

Since we are force enabling the mandatory tail calls on Windows x64, even at the expense of stack traces and debugging support, adjust the tests to account for the difference.

This is a cherry-pick of #63660

rdar://107919024

Since we are force enabling the mandatory tail calls on Windows x64,
even at the expense of stack traces and debugging support, adjust the
tests to account for the difference.
@eeckstein eeckstein requested a review from a team as a code owner April 12, 2023 05:47
@eeckstein eeckstein requested a review from compnerd April 12, 2023 05:47
@eeckstein
Copy link
Contributor Author

@swift-ci test

@eeckstein eeckstein changed the title test: adjust tests for mandatory tail calls on Win64 [5.9] test: adjust tests for mandatory tail calls on Win64 Apr 12, 2023
@eeckstein
Copy link
Contributor Author

@swift-ci test linux

@compnerd
Copy link
Member

Do we have the associated changes for this already cherry-picked to 5.9 for llvm?

@kavon
Copy link
Member

kavon commented Apr 12, 2023

merging to fix windows CI

@kavon
Copy link
Member

kavon commented Apr 12, 2023

@compnerd yes it appears so. these tests were failing because the compiler is now emitting the unreachable.

@kavon
Copy link
Member

kavon commented Apr 12, 2023

specifically, it landed as part of this swiftlang/llvm-project#6619 in this commit: swiftlang/llvm-project@67e69d5

@kavon kavon merged commit ce02630 into swiftlang:release/5.9 Apr 12, 2023
@compnerd
Copy link
Member

Awesome, didnt realize it was backported already.

@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 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.

4 participants