Skip to content

Shim intrinsics::atomic_singlethreadfence, etc.#1150

Merged
bors merged 2 commits into
rust-lang:masterfrom
divergentdave:patch-1
Jan 25, 2020
Merged

Shim intrinsics::atomic_singlethreadfence, etc.#1150
bors merged 2 commits into
rust-lang:masterfrom
divergentdave:patch-1

Conversation

@divergentdave

Copy link
Copy Markdown
Contributor

This applies the no-op shim for atomic fences to atomic_singlethreadfence and related intrinsics.

@oli-obk

oli-obk commented Jan 25, 2020

Copy link
Copy Markdown
Contributor

Can these be triggered from code using just the libstd? If so, please add a test invoking some of them

@divergentdave

Copy link
Copy Markdown
Contributor Author

👍 tests added

@RalfJung

Copy link
Copy Markdown
Member

Thanks!

@bors r+

@bors

bors commented Jan 25, 2020

Copy link
Copy Markdown
Contributor

📌 Commit 9265e0c has been approved by RalfJung

@bors

bors commented Jan 25, 2020

Copy link
Copy Markdown
Contributor

⌛ Testing commit 9265e0c with merge b222677...

bors added a commit that referenced this pull request Jan 25, 2020
Shim intrinsics::atomic_singlethreadfence, etc.

This applies the no-op shim for atomic fences to `atomic_singlethreadfence` and related intrinsics.
@bors

bors commented Jan 25, 2020

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing b222677 to master...

@bors
bors merged commit 9265e0c into rust-lang:master Jan 25, 2020
@divergentdave
divergentdave deleted the patch-1 branch January 25, 2020 20:43
@jonhoo

jonhoo commented Jan 27, 2020

Copy link
Copy Markdown

@RalfJung Can we submit a PR to get this into nightly? I would do it myself, but from memory last time it is easier if you are the owner of the PR in case further fixes are needed :)

@oli-obk

oli-obk commented Jan 27, 2020

Copy link
Copy Markdown
Contributor

Since miri is still passing on rustc master, you should be able to just open a PR that bumps the submodule and ping ralf and me so one of us can send it off. Most of the time we just update when there's breakage, so you may have seen that?

@jonhoo

jonhoo commented Jan 27, 2020

Copy link
Copy Markdown

Will do! I just remember the experience from rust-lang/rust#65278 (which was also "just a bump"), and wanted to avoid repeating that :p

@jonhoo

jonhoo commented Jan 27, 2020

Copy link
Copy Markdown

Done in rust-lang/rust#68576

bors pushed a commit to rust-lang/rust that referenced this pull request Jan 27, 2020
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants