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

Compile libFLAC with associative floating-point math #476

Merged
merged 6 commits into from
Oct 12, 2022

Conversation

ktmf01
Copy link
Collaborator

@ktmf01 ktmf01 commented Oct 9, 2022

This PR tries to improve like #473. I closed that one because it came to my attention linking with -ffast-math would result in 'infectious' flush-to-zero or denormals-are-zero behavior extending beyond the confines of libFLAC, affecting programs linking to libFLAC. See https://simonbyrne.github.io/notes/fastmath/#flushing_subnormals_to_zero

So, this PR achieves the same speed-up but without affecting programs linked against libFLAC.

@ktmf01 ktmf01 force-pushed the libFLAC-fast-math branch 2 times, most recently from b7c1999 to b18a655 Compare October 10, 2022 13:30
Also, fix a bug in which apodization windows were not recalculated when
blocksize was shrunk
@ktmf01 ktmf01 merged commit 90d7fdb into xiph:master Oct 12, 2022
@ktmf01 ktmf01 deleted the libFLAC-fast-math branch May 12, 2023 19:50
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.

None yet

1 participant