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

P0943 Support C atomics in C++ #365

Closed
wg21bot opened this issue Mar 20, 2019 · 11 comments · Fixed by cplusplus/draft#4372
Closed

P0943 Support C atomics in C++ #365

wg21bot opened this issue Mar 20, 2019 · 11 comments · Fixed by cplusplus/draft#4372
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e C++23 Targeted at C++23 IS Ship vehicle: IS plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. size - small paper size estimate
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Mar 20, 2019

P0943R2 Support C atomics in C++ (Hans-J. Boehm)

@wg21bot wg21bot added this to the 2019-07 milestone Mar 20, 2019
@wg21bot wg21bot added LEWG Library Evolution SG1 Concurrency labels Mar 20, 2019
@wg21bot
Copy link
Collaborator Author

wg21bot commented Jun 23, 2019

P0943R3 Support C atomics in C++ (Hans Boehm)

@ogiroux ogiroux added this to Monday in SG1 in Cologne 2019 Jul 13, 2019
@ogiroux ogiroux moved this from Monday to Thursday in SG1 in Cologne 2019 Jul 13, 2019
@ogiroux ogiroux moved this from Thursday to Monday in SG1 in Cologne 2019 Jul 13, 2019
@jensmaurer jensmaurer added this to Friday in LEWG in Cologne 2019 Jul 13, 2019
@tituswinters
Copy link

Prefer _Atomic instead of (something like) atomic_generic_type
SF F N A SA
4 5 5 2 1

Put <stdatomic.h> in Annex D as with the other C headers
SF F N A SA
1 3 6 4 0
(Don't put in Annex D).

Revise to be in terms of _Atomic, Forward to LWG for C++23.
SF F N A SA
3 11 2 0 0

@tituswinters tituswinters added C++23 Targeted at C++23 LWG Library and removed LEWG Library Evolution labels Jul 19, 2019
@ogiroux
Copy link
Collaborator

ogiroux commented Jul 19, 2019

(This note is out of order, it precedes the preceding note.)

Change 'atomic_generic_type' back to _Atomic
SF F N A SA
3 6 12 2 0 - Consensus

Paper proceeds without returning to SG1 if _Atomic is changed again
SF F N A SA
10 7 3 0 1 - Consensus

@ogiroux ogiroux removed the SG1 Concurrency label Jul 19, 2019
@wg21bot
Copy link
Collaborator Author

wg21bot commented Aug 23, 2019

P0943R4 Support C atomics in C++ (Hans Boehm)

@wg21bot wg21bot modified the milestones: 2019-07, 2019-11 Aug 23, 2019
@jensmaurer jensmaurer modified the milestones: 2019-11, 2020-06 Feb 18, 2020
@jensmaurer jensmaurer added IS Ship vehicle: IS and removed C++23 Targeted at C++23 labels Feb 18, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Mar 24, 2020

P0943R5 Support C atomics in C++ (Hans Boehm)

@JeffGarland JeffGarland added the lwg-prereview Needs LWG initial review before full group label Mar 24, 2020
@JeffGarland
Copy link
Member

Telecom review of paper with author on 2020-09-25. Significant revisions requested. Detailed notes on wiki.

https://wiki.edg.com/bin/view/Wg21summer2020/Telecon-20200925

@JeffGarland JeffGarland added lwg-fullreview Paper is ready for lwg full group review needs-revision Paper needs changes before it can proceed and removed lwg-prereview Needs LWG initial review before full group labels Sep 28, 2020
@ben-craig ben-craig added B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - small paper size estimate labels Oct 17, 2020
@JeffGarland
Copy link
Member

JeffGarland commented Oct 30, 2020

LWG telecon 2020-10-23 --

Wiki Notes: https://wiki.edg.com/bin/view/Wg21summer2020/D0943r6

Those in favor moving a [finished] version of P0943r6 in the November Plenary

F A N
10 0 0

@JeffGarland JeffGarland added tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed lwg-fullreview Paper is ready for lwg full group review labels Oct 30, 2020
@jensmaurer jensmaurer removed the needs-revision Paper needs changes before it can proceed label Nov 14, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Nov 23, 2020

P0943R6 Support C atomics in C++ (Hans Boehm)

@AaronBallman AaronBallman added the SG22 C / C++ liaison label Feb 10, 2021
@AaronBallman
Copy link
Collaborator

Reopening at the author's request so that SG22 can review the paper.

@AaronBallman AaronBallman reopened this Feb 10, 2021
@jwakely jwakely removed the tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. label Feb 12, 2021
@cor3ntin cor3ntin added C++23 Targeted at C++23 plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. and removed LWG Library labels Apr 22, 2021
@AaronBallman AaronBallman removed the SG22 C / C++ liaison label May 7, 2021
@AaronBallman
Copy link
Collaborator

Should _Atomic(T) and std::atomic have the same representation, size, alignment, and access protocol (e.g., lock free vs not, fence ordering, etc)?

SF F N A SA WG14
3 2 0 0 0 consensus
SF F N A SA WG21
4 3 0 0 0 consensus

The author will write a paper for WG14 to suggest adding the same recommended practice from P0943. Removing the SG22 tag as we don't expect to see an update for this paper and closing because this has already been applied for C++23.

@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

Adopted 2020-11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e C++23 Targeted at C++23 IS Ship vehicle: IS plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. size - small paper size estimate
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

9 participants