This is an archive of the discontinued LLVM Phabricator instance.

C++20 comments
Needs ReviewPublic

Authored by BenjaminTrapani on Nov 26 2020, 1:59 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Move constructors and assignment operators closer to top of commented decl

Clean up type names

Remove C11 atomic impl for std::atomic

Add error assertions

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptNov 26 2020, 1:59 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
BenjaminTrapani requested review of this revision.Nov 26 2020, 1:59 PM

Further rebases

BenjaminTrapani removed a reviewer: Restricted Project.Nov 26 2020, 2:23 PM
BenjaminTrapani removed rG LLVM Github Monorepo as the repository for this revision.
BenjaminTrapani removed a project: Restricted Project.
BenjaminTrapani removed subscribers: jfb, libcxx-commits.

Is this meant to be reviewed, or is this part of the atomic_ref patch? If the former, please abandon to clean up the review queue.