Skip to content

linker: Quote symbol names in .def files - #140505

Merged
bors merged 1 commit into
rust-lang:masterfrom
petrochenkov:expquote
May 3, 2025
Merged

linker: Quote symbol names in .def files#140505
bors merged 1 commit into
rust-lang:masterfrom
petrochenkov:expquote

Conversation

@petrochenkov

@petrochenkovpetrochenkov commented Apr 30, 2025

Copy link
Copy Markdown
Contributor

To support weird symbol names, including dots in particular.

cc #134767

@rustbot

Copy link
Copy Markdown
Collaborator

r? @nnethercote

rustbot has assigned @nnethercote.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Apr 30, 2025
@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in compiler/rustc_codegen_ssa

cc @WaffleLapkin

Comment threadcompiler/rustc_codegen_ssa/src/back/linker.rs
@bjorn3

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Apr 30, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 8298004 has been approved by bjorn3

It is now in the queue for this repository.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 30, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request May 1, 2025
linker: Quote symbol names in .def files
To support weird symbol names, including dots in particular.
cc [rust-lang#134767](rust-lang#134767 (comment))
bors added a commit to rust-lang-ci/rust that referenced this pull request May 1, 2025
…iaskrgr
Rollup of 7 pull requests
Successful merges:
- rust-lang#134034 (handle paren in macro expand for let-init-else expr)
- rust-lang#139186 (Refactor `diy_float`)
- rust-lang#140062 (std: mention `remove_dir_all` can emit `DirectoryNotEmpty` when concurrently written into)
- rust-lang#140430 (Improve test coverage of HIR pretty printing.)
- rust-lang#140485 (Optimize the codegen for `Span::from_expansion`)
- rust-lang#140505 (linker: Quote symbol names in .def files)
- rust-lang#140521 (interpret: better error message for out-of-bounds pointer arithmetic and accesses)
r? `@ghost`
`@rustbot` modify labels: rollup
Zalathar added a commit to Zalathar/rust that referenced this pull request May 1, 2025
linker: Quote symbol names in .def files
To support weird symbol names, including dots in particular.
cc [rust-lang#134767](rust-lang#134767 (comment))
@ZalatharZalathar mentioned this pull request May 1, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request May 1, 2025
Rollup of 11 pull requests
Successful merges:
- rust-lang#134034 (handle paren in macro expand for let-init-else expr)
- rust-lang#138703 (chore: remove redundant words in comment)
- rust-lang#139186 (Refactor `diy_float`)
- rust-lang#139343 (Change signature of File::try_lock and File::try_lock_shared)
- rust-lang#139780 (docs: Add example to `Iterator::take` with `by_ref`)
- rust-lang#139802 (Fix some grammar errors and hyperlinks in doc for `trait Allocator`)
- rust-lang#140034 (simd_select_bitmask: the 'padding' bits in the mask are just ignored)
- rust-lang#140062 (std: mention `remove_dir_all` can emit `DirectoryNotEmpty` when concurrently written into)
- rust-lang#140485 (Optimize the codegen for `Span::from_expansion`)
- rust-lang#140505 (linker: Quote symbol names in .def files)
- rust-lang#140521 (interpret: better error message for out-of-bounds pointer arithmetic and accesses)
r? `@ghost`
`@rustbot` modify labels: rollup
@Zalathar

Copy link
Copy Markdown
Member

Failed in rollup: #140541 (comment)

@bors r-

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels May 1, 2025
@jieyouxu

Copy link
Copy Markdown
Member

(BTW, there's //@ needs-crate-type: cdylib now for gating the target cdylib crate type support)

To support weird symbol names, including dots in particular.
@petrochenkov

Copy link
Copy Markdown
ContributorAuthor

@bors r=bjorn3

@bors

bors commented May 2, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 842858f has been approved by bjorn3

It is now in the queue for this repository.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels May 2, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request May 3, 2025
…iaskrgr
Rollup of 8 pull requests
Successful merges:
- rust-lang#139343 (Change signature of File::try_lock and File::try_lock_shared)
- rust-lang#140505 (linker: Quote symbol names in .def files)
- rust-lang#140534 (PassWrapper: adapt for llvm/llvm-project@f137c3d592e96330e450a8fd63ef…)
- rust-lang#140546 (Remove backtrace dep from anyhow in features status dump tool)
- rust-lang#140548 (Emit user type annotations for free consts in pattern position)
- rust-lang#140564 (Use present indicative tense in std::io::pipe() API docs)
- rust-lang#140568 (Add regression test for rust-lang#140545)
- rust-lang#140606 (Improve hir pretty printing)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit ca67f4d into rust-lang:masterMay 3, 2025
@rustbotrustbot added this to the 1.88.0 milestone May 3, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request May 3, 2025
Rollup merge of rust-lang#140505 - petrochenkov:expquote, r=bjorn3
linker: Quote symbol names in .def files
To support weird symbol names, including dots in particular.
cc [rust-lang#134767](rust-lang#134767 (comment))
@petrochenkov
petrochenkov deleted the expquote branch June 17, 2026 14:55
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@petrochenkov@rustbot@bjorn3@bors@Zalathar@jieyouxu@nnethercote