-
Notifications
You must be signed in to change notification settings - Fork 187
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is missing a very important thing: nowhere in here do we say whether it is legal or not for multiple atomic_ref objects of different scopes pointing to the same object to exist at the same time. We must be explicit on this, and I believe that we must forbid this, in case our handling of padding bits becomes "we zero them in a constructor of atomic_ref" (since that needs to occur atomically).
f72e6c2
to
9487989
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Wesley, I've left some comments.
Co-authored-by: gonzalobg <[email protected]>
Co-authored-by: Jake Hemstad <[email protected]>
…ics. Co-authored-by: gonzalobg <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final touches
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final touches.
No description provided.