-
Notifications
You must be signed in to change notification settings - Fork 199
Don't normalize *-unknown-windows-coff
as *-unknown-windows-msvc
#1558
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
Conversation
This fixes the inability to use `x86_64-unknown-windows-coff` and `i686-unknown-windows-coff` triples with Embedded Swift. *Steps to reproduce:* 1. Install latest development snapshot off `main`, e.g. `swift-DEVELOPMENT-SNAPSHOT-2024-03-01-a-osx`. 2. Try to build with `swiftc -target i686-unknown-windows-coff -enable-experimental-feature Embedded -wmo -c test.swift` *Expected result:* Code compiles successfully or an actionable diagnostic message is produced. *Actual result:* <unknown>:0: error: could not find module 'Swift' for target 'i686-unknown-windows-msvc'
@swift-ci test |
What would |
This would allow us to cross-compile to Windows from platforms other than Windows without requiring MSVC toolchain to be installed. |
Interesting. I'm not sure what the license agreement looks like for the separate Windows SDK download, but at least historically I would have been inclined to use the I'm not sure that I'd be interested to know what @compnerd thinks about this. (I'm relaxed about the change itself; it just raises some questions.) |
Parts of MinGW are licensed as GPL. I'm not interested in MinGW per se, only the approach it takes for cross-compilation to Windows.
This may be correct, but I don't think that justifies the current normalization approach where it defaults to |
Maybe. Maybe we should have a |
This fixes the inability to use
x86_64-unknown-windows-coff
andi686-unknown-windows-coff
triples with Embedded Swift.Steps to reproduce:
main
, e.g.swift-DEVELOPMENT-SNAPSHOT-2024-03-01-a-osx
.swiftc -target i686-unknown-windows-coff -enable-experimental-feature Embedded -wmo -c test.swift
Expected result:
Code compiles successfully or an actionable diagnostic message is produced.
Actual result: