Ensure aligned_alloc is in 32 byte chunks#84
Open
8W9aG wants to merge 2 commits intoanthonix:masterfrom
Open
Conversation
8W9aG
commented
Mar 2, 2021
8W9aG
commented
- The C++ standard does not define what happens if the size is not a multiple of the integral
- In macOS Big Sur this results in a NULL output
- This makes it impossible to use ffts_init_1d_real on this platform due to the size passed never being an integral of 32
* The C++ standard does not define what happens if the size is not a multiple of the integral * In macOS Big Sur this results in a NULL output * This makes it impossible to use ffts_init_1d_real on this platform due to the size passed never being an integral of 32
* Previous solution did not give multiples of 32
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.