MSVC targets should use COFF as their archive format #123467
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rustbot
added
A-testsuite
|
r? @wesleywiser |
This comment has been minimized.
This comment has been minimized.
dpaoliello
force-pushed
the
archcoff
branch
2 times, most recently
from
66d76fe
to
d95b6c1
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
FYI, successful Windows build in CI: https://github.com/rust-lang/rust/actions/runs/8560549847?pr=123467 |
|
These commits modify compiler targets. |
|
@rustbot label -T-infra |
rustbot
removed
the
T-infra
wesleywiser
reviewed
Apr 5, 2024
|
@bors r+ rollup |
bors
added
S-waiting-on-bors
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Apr 6, 2024
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#123294 (Require LLVM_CONFIG to be set in rustc_llvm/build.rs) - rust-lang#123467 (MSVC targets should use COFF as their archive format) - rust-lang#123498 (explaining `DefKind::Field`) - rust-lang#123519 (Improve cfg and check-cfg configuration) - rust-lang#123525 (CFI: Don't rewrite ty::Dynamic directly) - rust-lang#123526 (Do not ICE when calling incorrectly defined `transmute` intrinsic) - rust-lang#123528 (Hide async_gen_internals from standard library documentation) r? `@ghost` `@rustbot` modify labels: rollup
rust-timer
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Apr 6, 2024
Rollup merge of rust-lang#123467 - dpaoliello:archcoff, r=wesleywiser MSVC targets should use COFF as their archive format While adding support for Arm64EC I ran into an issue where the standard library's rlib was missing the "EC Symbol Table" which is required for the MSVC linker to find import library symbols (generated by Rust's `raw-dylib` feature) when building for EC. The root cause of the issue is that LLVM only generated symbol tables (including the EC Symbol Table) if the `ArchiveKind` is `COFF`, but the MSVC targets didn't set their archive format, so it was defaulting to GNU.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While adding support for Arm64EC I ran into an issue where the standard library's rlib was missing the "EC Symbol Table" which is required for the MSVC linker to find import library symbols (generated by Rust's
raw-dylibfeature) when building for EC.The root cause of the issue is that LLVM only generated symbol tables (including the EC Symbol Table) if the
ArchiveKindisCOFF, but the MSVC targets didn't set their archive format, so it was defaulting to GNU.