Skip to content

Add reduced crasher for SR-7958 #17103

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 2 commits into from
Jun 11, 2018
Merged

Add reduced crasher for SR-7958 #17103

merged 2 commits into from
Jun 11, 2018

Conversation

beccadax
Copy link
Contributor

Trying to use an argument label in an interpolation can cause various crashes. In swiftlang-1000.0.16.7, these are usually in SILGen; in master at 8f6028d, they’re in CSApply.

The file includes two samples. In swiftlang-1000.0.16.7, the first crashes in exactly the same place as the code snippet in SR-7958, while the second crashes elsewhere in SILGen. In master at 8f6028d, they both seem to crash in the same place.

Trying to use an argument label in an interpolation can cause various crashes. In swiftlang-1000.0.16.7, these are usually in SILGen; in master at 8f6028d, they’re in CSApply.
@beccadax
Copy link
Contributor Author

@CodaFi

@CodaFi
Copy link
Contributor

CodaFi commented Jun 11, 2018

LGTM. You can merge this first and rebase your fix PR after running the fix compiler crashers script or merge the fix first and run the script here.

@@ -0,0 +1,10 @@
// RUN: %target-swift-frontend -emit-silgen %s
Copy link
Contributor

@CodaFi CodaFi Jun 11, 2018

Choose a reason for hiding this comment

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

This should be

// RUN: not --crash %target-swift-frontend %s -emit-ir

@CodaFi
Copy link
Contributor

CodaFi commented Jun 11, 2018

One last nit and I'll take this.

@beccadax
Copy link
Contributor Author

@CodaFi Should be fixed now. I didn't realize crasher tests passed by still crashing!

Copy link
Contributor

@CodaFi CodaFi left a comment

Choose a reason for hiding this comment

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

⛵️

@CodaFi
Copy link
Contributor

CodaFi commented Jun 11, 2018

@swift-ci test and merge

@swift-ci swift-ci merged commit af428be into swiftlang:master Jun 11, 2018
@beccadax beccadax deleted the inarguable branch June 11, 2018 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants