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

Add LLVM based MinGW targets #669

Closed
wants to merge 1 commit into from

Conversation

mati865
Copy link
Contributor

@mati865 mati865 commented Mar 28, 2022

@dot-asm
Copy link
Contributor

dot-asm commented Mar 30, 2022

This is duplicate of comment at rust-lang

Not that my word weighs a lot here and I apologize, but I fail to see the need for an additional target. Is there a programming interface that clang uses/requires that gcc and/or Microsoft cl don't? What I'm trying to say is that it's rather the programming interface that defines any particular target platform, not the compiler. Compilers are supposed to be and actually are interchangeable. For example, on Windows you can use clang-cl instead of vendor cl with *-msvc targets. On Linux you can freely switch between clang and gcc. And I see no reason for why it would be impossible to have the similar arrangement for *-windows-gnu targets.

@dot-asm
Copy link
Contributor

dot-asm commented Jun 5, 2022

Just in case for reference, my remark was refuted in the referred thread at rust-lang. To summarize, it boils down to irreconcilable differences in exception handling mechanisms.

Copy link
Member

@thomcc thomcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm in favor of supporting the -gnullvm targets, but this seems to remove one of the -gnu targets?

src/lib.rs Show resolved Hide resolved
@thomcc thomcc added the O-windows Windows targets and toolchains label Oct 26, 2022
@mati865 mati865 closed this Oct 26, 2022
@mati865
Copy link
Contributor Author

mati865 commented Oct 26, 2022

Possibly due to repo move GitHub ignores pushes to this branch, moving to new PR: #734

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-windows Windows targets and toolchains
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants