Skip to content

[build] Silence -Wvoid-pointer-to-int-cast #538

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 1 commit into from
Mar 16, 2020

Conversation

bulbazord
Copy link
Contributor

libdispatch will fail to build with a newly introduced clang diagnostic
pointer-to-int-cast. libdispatch converts a void pointer to a
dispatch_invoke_flags_t (aka unsigned int) in a few places. Ideally we would not
be doing this, but this solution at least gets libdispatch building again
with a newer version of clang.

libdispatch will fail to build with a newly introduced clang diagnostic
`pointer-to-int-cast`. libdispatch converts a void pointer to a
dispatch_invoke_flags_t (aka unsigned int) in a few places. Ideally we would not
be doing this, but this solution at least gets libdispatch building again
with a newer version of clang.
@bulbazord bulbazord requested a review from compnerd February 18, 2020 21:10
@bulbazord
Copy link
Contributor Author

@swift-ci please test

@bulbazord
Copy link
Contributor Author

cc @compnerd mind looking at this when you get the chance?

@compnerd
Copy link
Member

LGTM; @ktopley-apple, okay to merge?

@CodaFi CodaFi requested a review from ktopley-apple February 29, 2020 01:19
@benlangmuir
Copy link
Contributor

@swift-ci please test

@benlangmuir benlangmuir merged commit 80b1772 into swiftlang:master Mar 16, 2020
@bulbazord bulbazord deleted the fix-build-new-flag branch March 16, 2020 21:10
rokhinip pushed a commit that referenced this pull request Nov 5, 2021
[build] Silence `-Wvoid-pointer-to-int-cast`

Signed-off-by: Rokhini Prabhu <rokhinip@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants