Skip to content

Revert portion of workaround landed in #1139 which disabled the mandatory perf optimization pass #1172

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 25, 2025

Conversation

stmontgomery
Copy link
Contributor

@stmontgomery stmontgomery commented Jun 25, 2025

This reverts most of the workaround I landed in #1139. A compiler fix in swiftlang/swift#82034 has resolved the problem that necessitated it, and our macOS CI is using a new-enough toolchain which has that fix.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@stmontgomery stmontgomery added this to the Swift 6.x milestone Jun 25, 2025
@stmontgomery stmontgomery self-assigned this Jun 25, 2025
@stmontgomery stmontgomery added enhancement New feature or request darwin 🍎 macOS, iOS, watchOS, tvOS, and visionOS support workaround Workaround for an issue in another component (may need to revert later) build 🧱 Affects the project's build configuration or process labels Jun 25, 2025
@stmontgomery
Copy link
Contributor Author

@swift-ci please test

@stmontgomery stmontgomery marked this pull request as draft June 25, 2025 03:33
@stmontgomery
Copy link
Contributor Author

CI results indicate the failure is still occurring, so need to investigate further.

…e there appears to be some new regression in compile-time constant detection
@stmontgomery
Copy link
Contributor Author

@swift-ci please test

@stmontgomery stmontgomery changed the title Revert "Work around macOS build failures affecting test targets (#1139)" Revert portion of workaround landed in #1139 which disabled the mandatory perf optimization pass Jun 25, 2025
@stmontgomery stmontgomery marked this pull request as ready for review June 25, 2025 21:29
@stmontgomery stmontgomery merged commit d305f59 into swiftlang:main Jun 25, 2025
3 checks passed
@stmontgomery stmontgomery deleted the revert-1139 branch June 25, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build 🧱 Affects the project's build configuration or process darwin 🍎 macOS, iOS, watchOS, tvOS, and visionOS support enhancement New feature or request workaround Workaround for an issue in another component (may need to revert later)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants