Skip to content
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

Fix compiler crash on Swift 5.2 when inlining test helper #1192

Merged
merged 2 commits into from
May 26, 2021

Conversation

Davidde94
Copy link
Contributor

Running the command

swift test -c release -Xswiftc -enable-testing --enable-test-discovery

On Linux with Swift 5.2 will result in a compiler crash. The bug in the compiler is to do with inlining a test helper. The solution is to make the helper never inline. This fixes the issue, so that the code is now testable on Swift 5.2.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 26, 2021

CLA Signed

The committers are authorized under a signed CLA.

Copy link
Collaborator

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a code comment to clarify why we're doing this?

@Davidde94
Copy link
Contributor Author

@Lukasa done :)

@Davidde94 Davidde94 requested a review from Lukasa May 26, 2021 12:09
@glbrntt glbrntt added the semver/none No version bump required. label May 26, 2021
@glbrntt glbrntt merged commit ec8e5e0 into grpc:main May 26, 2021
@Davidde94 Davidde94 deleted the de/fix-5.2-inline-bug branch May 27, 2021 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants