Skip to content

[FreeBSD] Fix FreeBSD build #5146

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

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Conversation

michael-yuji
Copy link
Member

Various fixes to enable building on FreeBSD.

Fixed an issue that warning messages from libthr can be emitted to stderr when thread specific data is used. This is due to CoreFoundation setting the TSD to CF_TSD_BAD_PTR even after PTHERAD_DESTRUCTOR_ITERATIONS iterators of dtor calls. This causes libthr to emit a warning to stderr about left-over non-null TSD after max destructor iterations.

Various fixes to enable building on FreeBSD.

Fixed an issue that warning messages from libthr can be emitted to stderr when
thread specific data is used. This is due to CoreFoundation setting the TSD to
`CF_TSD_BAD_PTR` even after `PTHERAD_DESTRUCTOR_ITERATIONS` iterators of dtor
calls. This causes libthr to emit a warning to stderr about left-over non-null
TSD after max destructor iterations.
@compnerd
Copy link
Member

compnerd commented Dec 9, 2024

@swift-ci please test

@michael-yuji
Copy link
Member Author

@swift-ci please test

1 similar comment
@compnerd
Copy link
Member

compnerd commented Dec 9, 2024

@swift-ci please test

@michael-yuji
Copy link
Member Author

@swift-ci please test Windows platform

@michael-yuji
Copy link
Member Author

@swift-ci please test

@parkera
Copy link
Contributor

parkera commented Dec 16, 2024

Are you ready to merge this?

@michael-yuji
Copy link
Member Author

Are you ready to merge this?

Yes, this pull request is ready to be merged. However, I lack write access to the repository.

@finagolfin finagolfin merged commit 238130a into swiftlang:main Dec 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants