Skip to content

Conversation

@grynspan
Copy link
Contributor

This PR changes the name of the clang module introduced in #83914 from _GUID to _GUIDDef. This change is necessary because the Windows SDK declares a type named _GUID and Swift gets confused when a module and a type have the same name.

This confusion is the motivation for SE-0491, but we can't just wait for a fix because the new module is a source-breaking change and clients won't be able to adopt the proposed syntax for some time.

This PR changes the name of the clang module introduced in #83914 from `_GUID`
to `_GUIDDef`. This change is necessary because the Windows SDK declares a type
named `_GUID` and Swift gets confused when a module and a type have the same
name.

This confusion is the motivation for [SE-0491](https://github.com/swiftlang/swift-evolution/blob/main/proposals/0491-module-selectors.md),
but we can't just wait for a fix because the new module is a source-breaking
change and clients won't be able to adopt the proposed syntax for some time.
@grynspan grynspan added this to the Swift 6.3 milestone Sep 23, 2025
@grynspan grynspan self-assigned this Sep 23, 2025
@grynspan grynspan requested review from a team and compnerd as code owners September 23, 2025 16:58
@grynspan grynspan added bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. Windows Platform: Windows module maps regression labels Sep 23, 2025
@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan grynspan enabled auto-merge (squash) September 23, 2025 20:49
@grynspan grynspan merged commit 4dd71a4 into main Sep 24, 2025
4 of 5 checks passed
@grynspan grynspan deleted the jgrynspan/rename-guid-module branch September 24, 2025 01:53
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Oct 6, 2025
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Oct 6, 2025
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Oct 6, 2025
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. module maps regression Windows Platform: Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants