Skip to content
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

abseil-cpp: Expose more missing headers #1864

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

ford-prefect
Copy link
Contributor

This is enough for webrtc-audio-processing-2 to compile.

This is enough for webrtc-audio-processing-2 to compile.
@ford-prefect
Copy link
Contributor Author

Thanks for the quick merge on the last one -- I missed a few and now tested this a bit better for the package this is being used in.

@neheb
Copy link
Collaborator

neheb commented Jan 10, 2025

know anything about that ppc failure?

@ford-prefect
Copy link
Contributor Author

know anything about that ppc failure?

Looking at this line, it seems abseil only supports PPC with glibc and not musl, at least for now.

@ford-prefect
Copy link
Contributor Author

If we want to, we could set ABSL_USE_UNSCALED_CYCLECLOCK to 0 to avoid building this on PPC, but I'm not sure what effects that would have, but it can't be too bad.

@neheb
Copy link
Collaborator

neheb commented Jan 10, 2025

Sure. I highly doubt anyone uses WrapDB on PPC anyway.

@ford-prefect ford-prefect force-pushed the absl-more-missing-headers branch from 0339d26 to 171a7c2 Compare January 10, 2025 16:15
@ford-prefect
Copy link
Contributor Author

Phew, looks like that got it to green.

@jpakkane jpakkane merged commit b71ad42 into mesonbuild:master Jan 10, 2025
12 checks passed
hw_cpp_flags = []
if host_machine.cpu_family() == 'x86_64'
hw_cpp_flags += ['-maes', '-msse4.1']
elif host_machine.cpu_family() == 'aarch64' and cpp.sizeof('void*') == 8
hw_cpp_flags += ['-march=armv8-a+crypto']
elif host_machine.cpu_family() == 'arm' and cpp.sizeof('void*') == 4
hw_cpp_flags += ['-mfpu=neon']
elif host_machine.cpu_family() == 'ppc' or host_machine.cpu_family() == 'ppc64'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

elif host_machine.cpu_family().startswith('ppc') or elif host_machine.cpu_family() in ['ppc', 'ppc64']
For future reference.

@ford-prefect ford-prefect deleted the absl-more-missing-headers branch January 10, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants