Skip to content

Check: Revert "Sema: Don't diagnose declarations more available than unavailable container." #64530

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

Closed
wants to merge 1 commit into from

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Mar 22, 2023

Check if #64515 is what caused the issues mentioned below in #64517

I'm concurrently investigating if any of my executor PRs could have caused this.

/Users/ec2-user/jenkins/workspace/swift-PR-macos/branch-main/swift/test/ModuleInterface/actor_availability.swift:95:15: warning: symbols that are @_spi_available on all platforms should use @_spi instead
public struct SPIAvailableStruct {
              ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos/branch-main/build/buildbot_incremental/swift-macosx-x86_64/test-iphonesimulator-x86_64/ModuleInterface/Output/actor_availability.swift.tmp/Library.swiftinterface:79:16: error: type 'SPIAvailableStruct.UnavailableNestedActor' does not conform to protocol 'Actor'
  public actor UnavailableNestedActor {
               ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos/branch-main/build/buildbot_incremental/swift-macosx-x86_64/test-iphonesimulator-x86_64/ModuleInterface/Output/actor_availability.swift.tmp/Library.swiftinterface:85:62: error: unavailable property 'unownedExecutor' was used to satisfy a requirement of protocol 'Actor'
    @_semantics("defaultActor") nonisolated final public var unownedExecutor: _Concurrency.UnownedSerialExecutor {
                                                             ^
_Concurrency.Actor:3:21: note: requirement 'unownedExecutor' declared here
    nonisolated var unownedExecutor: UnownedSerialExecutor { get }
                    ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos/branch-main/build/buildbot_incremental/swift-macosx-x86_64/test-iphonesimulator-x86_64/ModuleInterface/Output/actor_availability.swift.tmp/Library.swiftinterface:1:1: error: failed to verify module interface of 'Library' due to the errors above; the textual interface may be broken by project issues or a compiler bug
// swift-interface-format-version: 1.0
^

This reverts commit 620462d.

@ktoso
Copy link
Contributor Author

ktoso commented Mar 22, 2023

@swift-ci please test macOS

@ktoso ktoso requested a review from tshortli March 22, 2023 06:36
@ktoso
Copy link
Contributor Author

ktoso commented Mar 22, 2023

The Job not found issue; I'm still trying to figure it out -- going to PR a workaround for the tests...

@swift-ci please clean test macOS

@tshortli
Copy link
Contributor

Could you mark the test as unsupported on the platforms where it's failing, rather than reverting the whole change? I'll follow up to fix the test.

@ktoso
Copy link
Contributor Author

ktoso commented Mar 22, 2023

Okey I'll do that instead -- primarily wanted to see here if this was the change that caused the issues.
Folks assume it's my fault whenever anything 'executor' shows up in logs 🥲

I'll change this to disable the test and merge to give you time to follow up

@ktoso ktoso closed this Mar 22, 2023
@ktoso ktoso deleted the wip-fix-availability-test branch March 22, 2023 13:51
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