Skip to content

Conversation

cramertj
Copy link
Member

@cramertj cramertj commented Aug 7, 2018

This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.

cc @petrhosek

r? @alexcrichton

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 7, 2018
This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Aug 8, 2018

📌 Commit 877c469 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 8, 2018
cramertj added a commit to cramertj/rust that referenced this pull request Aug 8, 2018
Avoid increased alignment of TLS segments on Fuchsia

This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.

cc @petrhosek

r? @alexcrichton
cramertj added a commit to cramertj/rust that referenced this pull request Aug 8, 2018
Avoid increased alignment of TLS segments on Fuchsia

This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.

cc @petrhosek

r? @alexcrichton
cramertj added a commit to cramertj/rust that referenced this pull request Aug 8, 2018
Avoid increased alignment of TLS segments on Fuchsia

This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.

cc @petrhosek

r? @alexcrichton
cramertj added a commit to cramertj/rust that referenced this pull request Aug 8, 2018
Avoid increased alignment of TLS segments on Fuchsia

This is a temporary workaround for Fuchsia's libc not supporting
TLS segments with alignments greater than 32 bytes. It should
be reverted ASAP following the fix to libc.

cc @petrhosek

r? @alexcrichton
@cramertj
Copy link
Member Author

cramertj commented Aug 9, 2018

Our libc has a fix, so I get to back this out even before it lands XD

@cramertj cramertj closed this Aug 9, 2018
@cramertj cramertj deleted the tls-align branch August 9, 2018 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants