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

🍒 [5.6] backtrack on part of SE-327 dealing with default-value exprs #41967

Merged

Conversation

kavon
Copy link
Member

@kavon kavon commented Mar 23, 2022

This effectively reverts 6823744

The blanket removal of isolation in default-value expressions had
unintented consequences for important workflows. It's still
a problem that needs to be addressed, but we need to be more precise
about the problematic situations.

resolves rdar://90665119

This effectively reverts swiftlang@6823744

The blanket removal of isolation in default-value expressions had
unintented consequences for important workflows. It's still
a problem that needs to be addressed, but we need to be more precise
about the problematic situations.
@kavon kavon requested a review from a team as a code owner March 23, 2022 00:39
@kavon kavon added the r5.6 label Mar 23, 2022
@kavon
Copy link
Member Author

kavon commented Mar 23, 2022

@swift-ci please test

@kavon
Copy link
Member Author

kavon commented Mar 23, 2022

The macOS failure is a random CI machine issue.

Windows failure is probably unrelated, but I need to dig in to see why it broke. A cursory look suggests a change in the number of expected diagnostics, which could be due to the removal of warnings from this PR:

Input file: <stdin>

Check file: C:\Users\swift-ci\jenkins\workspace\swift-PR-windows\swift\test\ModuleInterface\ModuleCache\module-cache-diagnostics.swift



-dump-input=help explains the following input dump.



Input was:

<<<<<<

            1: BUILD_DIR\test-windows-x86_64\ModuleInterface\ModuleCache\Output\module-cache-diagnostics.swift.tmp\LeafModule.swiftinterface:7:8: error: no such module 'NotAModule' [] [] 

            2: Number FIXITs = 0 

            3: BUILD_DIR\test-windows-x86_64\ModuleInterface\ModuleCache\Output\module-cache-diagnostics.swift.tmp\OtherModule.swiftinterface:4:8: error: failed to build module 'LeafModule' for importation due to the errors above; the textual interface may be broken by project issues or a compiler bug [] [] 

check:61'0                                                                                                                                                                                                                                                                                                    X~~~~~~ error: no match found

            4: Number FIXITs = 0 

check:61'0     ~~~~~~~~~~~~~~~~~~

            5: BUILD_DIR\test-windows-x86_64\ModuleInterface\ModuleCache\Output\module-cache-diagnostics.swift.tmp\LeafModule.swiftinterface:98:8: error: failed to build module 'OtherModule' for importation due to the errors above; the textual interface may be broken by project issues or a compiler bug [] [] 

check:61'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

check:61'1                                                                                             ?                                                                                                                                                                                                               possible intended match

            6: Number FIXITs = 0 

check:61'0     ~~~~~~~~~~~~~~~~~~

            7: Number of diagnostics: 3 

check:61'0     ~~~~~~~~~~~~~~~~~~~~~~~~~

>>>>>>


error: command failed with exit status: 1

@kavon
Copy link
Member Author

kavon commented Mar 23, 2022

@swift-ci please test Windows

@kavon
Copy link
Member Author

kavon commented Mar 23, 2022

@swift-ci please test macOS

@DougGregor DougGregor merged commit 7c5b1aa into swiftlang:release/5.6 Mar 23, 2022
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.6 labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants