Skip to content

Conversation

…irectory

They're actually fast.

Also, change -solver-expression-time-threshold=1 into a scope limit to make it
deterministic.
@slavapestov
Copy link
Contributor Author

@swift-ci Please smoke test

@slavapestov slavapestov enabled auto-merge October 14, 2025 18:46
@slavapestov
Copy link
Contributor Author

@swift-ci Please test Windows

@slavapestov
Copy link
Contributor Author

@swift-ci Please smoke test macOS

Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

// RUN: %target-typecheck-verify-swift -solver-scope-threshold=300

func slow(someOptionalString: String?) {
print("\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")\(someOptionalString ?? "")")
Copy link
Contributor

Choose a reason for hiding this comment

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

I vaguely remember one with a bunch of ?? operators like that in the suite, could you please double-check if we already have it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We had a bunch with ?? but this one also has string interpolation. Do you think it’s worth keeping?

Copy link
Contributor

Choose a reason for hiding this comment

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

I see, sure!

@slavapestov slavapestov merged commit 0a4b6f0 into swiftlang:main Oct 15, 2025
3 checks passed
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.

2 participants