Skip to content

Include <stdnoreturn.h> last #1330

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

Conversation

Steelskin
Copy link
Contributor

Starting with the Windows SDK 10.0.26100.3916, inclusion of <stdnoreturn.h> clashes with some of the definitions in other ucrt headers. Move <stdnoreturn.h> to be last to work around this issue.

Starting with the Windows SDK 10.0.26100.3916, inclusion of
`<stdnoreturn.h>` clashes with some of the definitions in other ucrt
headers. Move `<stdnoreturn.h>` to be last to work around this issue.
@compnerd
Copy link
Member

compnerd commented Jun 3, 2025

:sigh: this is horrible ...

#define noreturn _Noreturn
__declspec(noreturn)

is then translated into __declspec(_Noreturn) which is not valid :/

@compnerd
Copy link
Member

compnerd commented Jun 3, 2025

@swift-ci please test

@compnerd compnerd requested a review from jmschonfeld June 5, 2025 17:06
@compnerd
Copy link
Member

compnerd commented Jun 5, 2025

Going to merge this - this should be harmless.

@compnerd compnerd merged commit 22cac91 into swiftlang:main Jun 5, 2025
15 checks passed
@Steelskin Steelskin deleted the fabrice/fix-header-inclue-sdk-26100 branch June 5, 2025 17:14
Steelskin added a commit to Steelskin/swift-foundation that referenced this pull request Jun 5, 2025
Starting with the Windows SDK 10.0.26100.3916, inclusion of
`<stdnoreturn.h>` clashes with some of the definitions in other ucrt
headers. Move `<stdnoreturn.h>` to be last to work around this issue.
chloe-yeo pushed a commit to chloe-yeo/swift-foundation that referenced this pull request Jun 10, 2025
Starting with the Windows SDK 10.0.26100.3916, inclusion of
`<stdnoreturn.h>` clashes with some of the definitions in other ucrt
headers. Move `<stdnoreturn.h>` to be last to work around this issue.
chloe-yeo pushed a commit to chloe-yeo/swift-foundation that referenced this pull request Jun 16, 2025
Starting with the Windows SDK 10.0.26100.3916, inclusion of
`<stdnoreturn.h>` clashes with some of the definitions in other ucrt
headers. Move `<stdnoreturn.h>` to be last to work around this issue.
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