Skip to content
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.

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

Extend the atomic ordering documentation. #40306

Closed
ticki opened this issue Mar 6, 2017 · 4 comments
Closed

Extend the atomic ordering documentation. #40306

ticki opened this issue Mar 6, 2017 · 4 comments
Labels
A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools C-enhancement Category: An issue proposing an enhancement or a PR with one. E-hard Call for participation: Hard difficulty. Experience needed to fix: A lot. P-medium Medium priority

Comments

@ticki
Copy link
Contributor

ticki commented Mar 6, 2017

The LLVM docs explains is pretty well, but it would be handy if the Rust docs were more detailed.

I'm not sure about the license of the LLVM docs.

tag @steveklabnik

@ghost
Copy link

ghost commented Mar 7, 2017

There's also quite a bit of detail in the Atomics section of Rustonomicon.

@steveklabnik steveklabnik added A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools P-medium Medium priority and removed A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools labels Mar 10, 2017
@steveklabnik
Copy link
Member

docs team triage: p-medium

@projektir projektir mentioned this issue May 15, 2017
31 tasks
@Mark-Simulacrum Mark-Simulacrum added the C-enhancement Category: An issue proposing an enhancement or a PR with one. label Jul 27, 2017
@steveklabnik steveklabnik added E-medium Call for participation: Medium difficulty. Experience needed to fix: Intermediate. E-hard Call for participation: Hard difficulty. Experience needed to fix: A lot. and removed E-medium Call for participation: Medium difficulty. Experience needed to fix: Intermediate. labels May 29, 2018
@DevQps
Copy link
Contributor

DevQps commented Apr 5, 2019

@ticki I was browsing around a bit and saw the std::sync::atomic::Ordering now does quite a good job at explaining the different types of atomic orderings! It even links to the LLVM pages that you provided as well and matches the displayed Ordering with the associated LLVM ones. As has been mentioned by the deleted user, the nomicon also contains some good documentation as well, which seems to be more in depth.

@steveklabnik @ticki I think if you all agree we can close this one now!

@steveklabnik
Copy link
Member

Ticki doens't do open source anymore, and yeah. Without specific improvements suggested, and both other people in this thread no longer around, I think we can close. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools C-enhancement Category: An issue proposing an enhancement or a PR with one. E-hard Call for participation: Hard difficulty. Experience needed to fix: A lot. P-medium Medium priority
Projects
None yet
Development

No branches or pull requests

4 participants