-
Notifications
You must be signed in to change notification settings - Fork 8
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
Make not working on Mojave #26
Comments
This is the same compilation error that used to appear in Travis' macOS builds, even on High Sierra. I fixed it via f623ae8, which I found from some other GitHub repository feeling the same effects. I've never experienced this error on either of my macs, but look forward to it! |
Also, the default |
The FFT time is non-negligible, right? Why not use MKL which is apparently much faster? |
Could be used, but would require some modifications since MKL is incomplete. In particular, it doesn't implement real-to-complex transforms with split storage, halfcomplex-to-real transforms, or multidimensional real-to-real transforms. https://software.intel.com/en-us/mkl-developer-reference-c-using-fftw3-wrappers |
BTW, I added a proposal for Julia wrapper here JuliaApproximation/FastTransforms.jl@ee98f40 |
Fixed it!! Here's the instructions
to restore
to Make.inc |
Is 2. necessary? are they not default search paths for gcc (which is why it can't find the headers to begin with)? |
Probably: I tried without them. |
Fun fact that I've been experimenting with recently: you can compile one file into assembly using one compiler (gcc-8 on rotations.c => rotations.s) and then use a different compiler to turn the assembly into executables/libraries. Clang has significantly better control over OpenMP threads, which is pronounced for low degrees. |
Actually, I just needed to rebuild gcc (I deleted /usr/local and started from scratch with homebrew) |
The text was updated successfully, but these errors were encountered: