Add #[inline] back on __getit (fixes #25088)#50252
Add #[inline] back on __getit (fixes #25088)#50252nox wants to merge 1 commit intorust-lang:masterfrom
Conversation
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
@bors r+ (let's see if it passes on windows) |
|
📌 Commit 89f3970 has been approved by |
|
The rollup #50275 failed on Given the previous test result, I think this PR is the cause. |
|
@kennytm Sorry, this should have been excluded from rollup, because we were checking for that failure - is there any way for someone to indicate that a PR is "standalone" yet? |
"(Do not merge)" as a prefix of the PR title would probably signal that this shouldn't go in a rollup. |
|
@nox |
|
If this is claiming to fix #25088 can a codegen test be added to assert such functionality? This has historically been very finnicky and easy to regress. |
|
⌛ Testing commit 89f3970 with merge ed3ce2ec4df3acb3efcec3d97a0a9876f6aadcf2... |
|
💔 Test failed - status-appveyor |
|
Same error as #50252 (comment). @bors r- |
|
Thank you for this PR @nox! Unfortunately we haven't heard from you on this in a while, so I'm closing the PR to keep things tidy. Don't worry though, if you'll have time again in the future please reopen this PR, we'll be happy to review it again! |
Issue rust-lang#25088 has been part of `thread_local!` for quite some time now. Historical attempts have been made to add `#[inline]` to `__getit` in rust-lang#43931, rust-lang#50252, and rust-lang#59720, but these attempts ended up not landing at the time due to segfaults on Windows. In the interim though with `const`-initialized thread locals AFAIK this is the only remaining bug which is why you might want to use `#[thread_local]` over `thread_local!`. As a result I figured it was time to resubmit this and see how it fares on CI and if I can help debugging any issues that crop up. Closes rust-lang#25088
…ss-crate, r=Mark-Simulacrum std: Attempt again to inline thread-local-init across crates Issue rust-lang#25088 has been part of `thread_local!` for quite some time now. Historical attempts have been made to add `#[inline]` to `__getit` in rust-lang#43931, rust-lang#50252, and rust-lang#59720, but these attempts ended up not landing at the time due to segfaults on Windows. In the interim though with `const`-initialized thread locals AFAIK this is the only remaining bug which is why you might want to use `#[thread_local]` over `thread_local!`. As a result I figured it was time to resubmit this and see how it fares on CI and if I can help debugging any issues that crop up. Closes rust-lang#25088
No description provided.