Skip to content

Conversation

@Bryntet
Copy link
Contributor

@Bryntet Bryntet commented Dec 20, 2025

Simple port, only thing that might be an issue is the help: #[thread_local] can be applied to foreign statics and statics it probably would be that if an attribute is applicable to both statics and foreign statics that we don't alphabetically sort them next to each other, because now it kinda reads like #[thread_local] is something that you primarily use on foreign statics

r? @JonathanBrouwer

@rustbot
Copy link
Collaborator

rustbot commented Dec 20, 2025

Some changes occurred in compiler/rustc_hir/src/attrs

cc @jdonszelmann

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

cc @jdonszelmann

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

@rustbot rustbot added the A-attributes Area: Attributes (`#[…]`, `#![…]`) label Dec 20, 2025
@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. labels Dec 20, 2025
@JonathanBrouwer
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 20, 2025

📌 Commit 120f0d4 has been approved by JonathanBrouwer

It is now in the queue for this repository.

@bors bors 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 Dec 20, 2025
bors added a commit that referenced this pull request Dec 20, 2025
…uwer

Rollup of 2 pull requests

Successful merges:

 - #150188 (Dogfood `strip_circumfix`)
 - #150198 (Port `#[thread_local]` to attribute parser)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 73292d9 into rust-lang:main Dec 20, 2025
11 checks passed
rust-timer added a commit that referenced this pull request Dec 20, 2025
Rollup merge of #150198 - Bryntet:parse_thread_local, r=JonathanBrouwer

Port `#[thread_local]` to attribute parser

Simple port, only thing that might be an issue is the `help: #[thread_local] can be applied to foreign statics and statics` it probably would be that if an attribute is applicable to both statics and foreign statics that we don't alphabetically sort them next to each other, because now it kinda reads like `#[thread_local]` is something that you primarily use on foreign statics

r? `@JonathanBrouwer`
@rustbot rustbot added this to the 1.94.0 milestone Dec 20, 2025
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Dec 21, 2025
…uwer

Rollup of 2 pull requests

Successful merges:

 - rust-lang/rust#150188 (Dogfood `strip_circumfix`)
 - rust-lang/rust#150198 (Port `#[thread_local]` to attribute parser)

r? `@ghost`
`@rustbot` modify labels: rollup
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.

4 participants