Skip to content

Move proc-macro dlopen from proc-macro-srv to rustc_metadata#157711

Merged
rust-bors[bot] merged 3 commits into
rust-lang:mainfrom
bjorn3:proc_macro_refactor7
Jun 22, 2026
Merged

Move proc-macro dlopen from proc-macro-srv to rustc_metadata#157711
rust-bors[bot] merged 3 commits into
rust-lang:mainfrom
bjorn3:proc_macro_refactor7

Conversation

@bjorn3

@bjorn3 bjorn3 commented Jun 10, 2026

Copy link
Copy Markdown
Member

This will make it easier to change how proc-macros are implemented on the rustc side. For example allowing support for wasm proc-macros without duplicating the implementation between proc-macro-srv and rustc.

bjorn3 added 3 commits June 10, 2026 16:45
This will make it easier to change how proc-macros are implemented on
the rustc side. For example allowing support for wasm proc-macros
without duplicating the implementation between proc-macro-srv and rustc.
@rustbot

rustbot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

@rustbot rustbot 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. T-rust-analyzer Relevant to the rust-analyzer team, which will review and decide on the PR/issue. labels Jun 10, 2026
@rustbot

rustbot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Failed to set assignee to nikomatsakis: req=https://api.github.com/repos/rust-lang/rust/issues/157711/assignees (x-github-request-id: 4E39:7CA7:8FA626E:9109204:6A297E53): {
"message": "Requires authentication",
"documentation_url": "https://docs.github.com/rest",
"status": "401"
}

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@rustbot

rustbot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

r? @nikomatsakis

rustbot has assigned @nikomatsakis.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 73 candidates
  • Random selection from 17 candidates

@bjorn3 bjorn3 requested review from rcvalle and removed request for rcvalle June 10, 2026 15:10
@bjorn3

bjorn3 commented Jun 10, 2026

Copy link
Copy Markdown
Member Author

(accidentally did a review request as github decided to move some ui elements right when I clicked)

@Veykril Veykril left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

rust-analyze side

View changes since this review

@bjorn3

bjorn3 commented Jun 20, 2026

Copy link
Copy Markdown
Member Author

Gentle ping @nikomatsakis.

@Mark-Simulacrum

Copy link
Copy Markdown
Member

I can take this since I've been poking at this already.

r? me

@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+

This seems reasonable. Thanks!

@rust-bors

rust-bors Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 88c2f7d has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 Jun 21, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 21, 2026
…k-Simulacrum

Move proc-macro dlopen from proc-macro-srv to rustc_metadata

This will make it easier to change how proc-macros are implemented on the rustc side. For example allowing support for wasm proc-macros without duplicating the implementation between proc-macro-srv and rustc.
rust-bors Bot pushed a commit that referenced this pull request Jun 21, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - #156356 (bootstrap: add bootstrap step to run intrinsic-test in CI)
 - #157711 (Move proc-macro dlopen from proc-macro-srv to rustc_metadata)
 - #157836 (rebuild LLVM when `bootstrap.toml` config changes)
 - #158214 (Don't try to remove assignments in SimplifyComparisonIntegral)
 - #158226 (miri subtree update)
 - #158108 (Update actions/download-artifact action to v8.0.1)
 - #158150 (Update backtrace submodule to pick up AIX related fixes.)
 - #158178 (Use the target checking infrastructure for the diagnostic attributes)
 - #158195 (Add me to some rotation groups)
@rust-bors rust-bors Bot merged commit b0f7e12 into rust-lang:main Jun 22, 2026
12 checks passed
@rustbot rustbot added this to the 1.98.0 milestone Jun 22, 2026
rust-timer added a commit that referenced this pull request Jun 22, 2026
Rollup merge of #157711 - bjorn3:proc_macro_refactor7, r=Mark-Simulacrum

Move proc-macro dlopen from proc-macro-srv to rustc_metadata

This will make it easier to change how proc-macros are implemented on the rustc side. For example allowing support for wasm proc-macros without duplicating the implementation between proc-macro-srv and rustc.
@bjorn3 bjorn3 deleted the proc_macro_refactor7 branch June 22, 2026 08:53
pull Bot pushed a commit to xtqqczze/rust-lang-miri that referenced this pull request Jun 22, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - rust-lang/rust#156356 (bootstrap: add bootstrap step to run intrinsic-test in CI)
 - rust-lang/rust#157711 (Move proc-macro dlopen from proc-macro-srv to rustc_metadata)
 - rust-lang/rust#157836 (rebuild LLVM when `bootstrap.toml` config changes)
 - rust-lang/rust#158214 (Don't try to remove assignments in SimplifyComparisonIntegral)
 - rust-lang/rust#158226 (miri subtree update)
 - rust-lang/rust#158108 (Update actions/download-artifact action to v8.0.1)
 - rust-lang/rust#158150 (Update backtrace submodule to pick up AIX related fixes.)
 - rust-lang/rust#158178 (Use the target checking infrastructure for the diagnostic attributes)
 - rust-lang/rust#158195 (Add me to some rotation groups)
pull Bot pushed a commit to p-avital/rust-analyzer that referenced this pull request Jun 22, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - rust-lang/rust#156356 (bootstrap: add bootstrap step to run intrinsic-test in CI)
 - rust-lang/rust#157711 (Move proc-macro dlopen from proc-macro-srv to rustc_metadata)
 - rust-lang/rust#157836 (rebuild LLVM when `bootstrap.toml` config changes)
 - rust-lang/rust#158214 (Don't try to remove assignments in SimplifyComparisonIntegral)
 - rust-lang/rust#158226 (miri subtree update)
 - rust-lang/rust#158108 (Update actions/download-artifact action to v8.0.1)
 - rust-lang/rust#158150 (Update backtrace submodule to pick up AIX related fixes.)
 - rust-lang/rust#158178 (Use the target checking infrastructure for the diagnostic attributes)
 - rust-lang/rust#158195 (Add me to some rotation groups)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants