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.

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

Second sentence of 1.10p6 is redundant #159

Closed
hboehm opened this issue Jul 31, 2013 · 2 comments · Fixed by #990
Closed

Second sentence of 1.10p6 is redundant #159

hboehm opened this issue Jul 31, 2013 · 2 comments · Fixed by #990

Comments

@hboehm
Copy link

hboehm commented Jul 31, 2013

Please delete

"If A and B are modifications of an atomic object M and A happens before (as defined below) B, then A shall precede B in the modification order of M, which is defined below. [ Note: This states that the modification orders must respect the “happens before” relationship. —end note ]"

from 1.10p6 [intro.multithread]. The normative sentence is redundant with 1.10p15. There seems to be a concensus among SG1 commentators that it's not worth preserving the note, which seems a bit shallow. It could also be added to p15 or integrated into p19.
@hboehm
Copy link
Author

hboehm commented Jul 31, 2013

We did have one dissenting vote from Ville for keeping the note. The other 4 opinions seemed to weakly favor dropping it.

@jensmaurer
Copy link
Member

This is now 1.10.1 [intro.races] p4 and p13.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 12, 2016
@jensmaurer
[intro.races] remove redundant constraint on modification order
eed91bc 
fixes cplusplus#159
jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 12, 2016
@jensmaurer
[intro.races] remove redundant constraint on modification order
bf05d04 
fixes cplusplus#159
@jensmaurer jensmaurer mentioned this issue Nov 12, 2016
Merged
zygoloid pushed a commit that referenced this issue Nov 12, 2016
@jensmaurer @zygoloid
[intro.races] remove redundant constraint on modification order (#990)
4bcaad2 
Fixes #159
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[intro.races] remove redundant constraint on modification order jensmaurer/draft
2 participants
@hboehm @jensmaurer