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

Embedded: enable *-unknown-windows-coff triples in Platform.cpp #72217

Merged
merged 1 commit into from Mar 11, 2024

Conversation

MaxDesiatov
Copy link
Member

This allows the frontend to proceed when building with -target i686-unknown-windows-coff -enable-experimental-feature Embedded arguments.

This allows the frontend to proceed when building with `-target i686-unknown-windows-coff -enable-experimental-feature Embedded` arguments.
@MaxDesiatov MaxDesiatov added Windows Platform: Windows Frontend Area → compiler: Frontend embedded Embedded Swift labels Mar 10, 2024
@MaxDesiatov MaxDesiatov self-assigned this Mar 10, 2024
@MaxDesiatov
Copy link
Member Author

@swift-ci smoke test

@MaxDesiatov MaxDesiatov added the needs tests Flag: Needs tests label Mar 10, 2024
@al45tair
Copy link
Contributor

What does it mean to have no environment on Windows? (See also apple/swift-driver#1558.)

@MaxDesiatov
Copy link
Member Author

MaxDesiatov commented Mar 11, 2024

What does it mean to have no environment on Windows? (See also apple/swift-driver#1558.)

IIUC it means that no assumptions about current environment are made and MSVC toolchain is not required. One could use MinGW infrastructure or something else to build for this triple.

@MaxDesiatov
Copy link
Member Author

@swift-ci test macos

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) March 11, 2024 11:22
@al45tair
Copy link
Contributor

One could use MinGW infrastructure or something else to build for this triple.

Let's discuss that on the other PR. I have no objection to supporting the -coff triple — I just want to know what it means and why we'd use it rather than e.g. -mingw, -cygwin or -msvc.

@MaxDesiatov MaxDesiatov merged commit 321cc83 into main Mar 11, 2024
3 of 4 checks passed
@MaxDesiatov MaxDesiatov deleted the maxd/embedded-coff-frontend branch March 11, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embedded Embedded Swift Frontend Area → compiler: Frontend needs tests Flag: Needs tests Windows Platform: Windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants