Skip to content

Conversation

@Ozzy1423
Copy link
Contributor

Tracking issue: #131229

Ports compiler_builtins, panic_runtime, needs_panic_runtime, profiler_runtime and no_builtins to the attribute parsing infrastructure.

r? @JonathanBrouwer

@rustbot
Copy link
Collaborator

rustbot commented Jan 18, 2026

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann, @JonathanBrouwer

Some changes occurred in compiler/rustc_hir/src/attrs

cc @jdonszelmann, @JonathanBrouwer

Some changes occurred in compiler/rustc_passes/src/check_attr.rs

cc @jdonszelmann, @JonathanBrouwer

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) 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 Jan 18, 2026
@rust-bors

This comment has been minimized.

Copy link
Contributor

@JonathanBrouwer JonathanBrouwer left a comment

Choose a reason for hiding this comment

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

r=me after rebasing

View changes since this review

@rustbot

This comment has been minimized.

@JonathanBrouwer
Copy link
Contributor

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 19, 2026

📌 Commit b755b3b has been approved by JonathanBrouwer

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. 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-review Status: Awaiting review from the assignee but also interested parties. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 19, 2026
@rust-bors

This comment has been minimized.

@rustbot

This comment has been minimized.

@JonathanBrouwer
Copy link
Contributor

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 20, 2026

📌 Commit 37d505c has been approved by JonathanBrouwer

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 20, 2026
rust-bors bot pushed a commit that referenced this pull request Jan 21, 2026
Rollup of 6 pull requests

Successful merges:

 - #150436 (`c_variadic`: impl `va_copy` and `va_end` as Rust intrinsics)
 - #151340 (Port `#[patchable_function_entry]` to attr parser)
 - #151351 (Deduplicate diagnostics for const trait supertraits)
 - #151424 (missing colon after the compile-flags directive)
 - #151428 (Port variance attrs to attr parser.)
 - #151429 (s390x: Support aligned stack datalayout)

Failed merges:

 - #151343 (Port some crate level attrs to the attribute parser)

r? @ghost
@rust-bors rust-bors bot added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jan 21, 2026
@rust-bors

This comment has been minimized.

@rust-bors rust-bors bot removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 21, 2026
@rustbot
Copy link
Collaborator

rustbot commented Jan 21, 2026

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@Ozzy1423
Copy link
Contributor Author

I'll wait for this #151442 and then rebase a final time!

@JonathanBrouwer
Copy link
Contributor

JonathanBrouwer commented Jan 21, 2026

I'll let this one go first, crate_type still needs a perf run
@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 21, 2026

📌 Commit f6d7638 has been approved by JonathanBrouwer

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 21, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jan 21, 2026
Port some crate level attrs to the attribute parser

Tracking issue: rust-lang#131229

Ports compiler_builtins, panic_runtime, needs_panic_runtime, profiler_runtime and no_builtins to the attribute parsing infrastructure.

r? @JonathanBrouwer
rust-bors bot pushed a commit that referenced this pull request Jan 21, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #151010 (std: use `ByteStr`'s `Display` for `OsStr`)
 - #151156 (Add GCC and the GCC codegen backend to build-manifest and rustup)
 - #151219 (Fixed ICE when using function pointer as const generic parameter)
 - #151343 (Port some crate level attrs to the attribute parser)
 - #151463 (add x86_64-unknown-linux-gnuasan to CI)

r? @ghost
@rust-bors rust-bors bot merged commit 18504a1 into rust-lang:main Jan 22, 2026
11 checks passed
@rustbot rustbot added this to the 1.95.0 milestone Jan 22, 2026
rust-timer added a commit that referenced this pull request Jan 22, 2026
Rollup merge of #151343 - Ozzy1423:attrs2, r=JonathanBrouwer

Port some crate level attrs to the attribute parser

Tracking issue: #131229

Ports compiler_builtins, panic_runtime, needs_panic_runtime, profiler_runtime and no_builtins to the attribute parsing infrastructure.

r? @JonathanBrouwer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants