Skip to content

fix alignment of pthread_attr_t #10305

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

fix alignment of pthread_attr_t #10305

wants to merge 1 commit into from

Conversation

thestinger
Copy link
Contributor

Closes #10300

@alexcrichton
Copy link
Member

Is there a way to get alignment in C/C++? If so, it seems like we should have a unit test which asserts the alignment about structures like this.

@thestinger
Copy link
Contributor Author

In C11/C++11, there's alignof. Neither clang or gcc seems to implement it for -std=c11 yet, just -std=c++11.

bors added a commit that referenced this pull request Nov 6, 2013
@bors
auto merge of #10305 : thestinger/rust/align, r=alexcrichton
efaf730 
Closes #10300
@bors bors closed this Nov 6, 2013
@thestinger thestinger deleted the align branch November 9, 2013 05:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pthread_attr_t definition is incorrect, will cause unaligned memory accesses
3 participants
@thestinger @alexcrichton @bors