Skip to content

[beta] Revert: Make TLS __getit #[inline(always)] on non-Windows #96138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 17, 2022
Merged

[beta] Revert: Make TLS __getit #[inline(always)] on non-Windows #96138

merged 1 commit into from
Apr 17, 2022

Conversation

erikdesjardins
Copy link
Contributor

@erikdesjardins
Revert "Auto merge of rust-lang#94373 - erikdesjardins:getitinl, r=Ma…
6a8a8f5 
…rk-Simulacrum"

This reverts commit 035a717, reversing
changes made to 761e888.
@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Apr 17, 2022
@rust-highfive
Copy link
Contributor

Hey! It looks like you've submitted a new PR for the library teams!

If this PR contains changes to any rust-lang/rust public library APIs then please comment with r? rust-lang/libs-api @rustbot label +T-libs-api -T-libs to request review from a libs-api team reviewer. If you're unsure where your change falls no worries, just leave it as is and the reviewer will take a look and make a decision to forward on if necessary.

Examples of T-libs-api changes:

  • Stabilizing library features
  • Introducing insta-stable changes such as new implementations of existing stable traits on existing stable types
  • Introducing new or changing existing unstable library APIs (excluding permanently unstable features / features without a tracking issue)
  • Changing public documentation in ways that create new stability guarantees
  • Changing observable runtime behavior of library APIs

@rust-highfive
Copy link
Contributor

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 17, 2022
@Mark-Simulacrum
Copy link
Member

@bors r+ rollup=never

@bors
Copy link
Collaborator

bors commented Apr 17, 2022

📌 Commit 6a8a8f5 has been approved by Mark-Simulacrum

@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 Apr 17, 2022
@bors
Copy link
Collaborator

bors commented Apr 17, 2022

⌛ Testing commit 6a8a8f5 with merge 2431a97...

@bors
Copy link
Collaborator

bors commented Apr 17, 2022

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing 2431a97 to beta...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 17, 2022
@bors bors merged commit 2431a97 into rust-lang:beta Apr 17, 2022
@rustbot rustbot added this to the 1.61.0 milestone Apr 17, 2022
@erikdesjardins erikdesjardins deleted the revertinl branch April 18, 2022 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Mark-Simulacrum Mark-Simulacrum

Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.61.0
Development

Successfully merging this pull request may close these issues.
5 participants
@erikdesjardins @rust-highfive @Mark-Simulacrum @bors @rustbot