-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Tests failing with USE_MKL #3902
Comments
Confirmed. ARPACK worker silently dies, amidst a sea of library symbol conflict warnings. I won't be able to look into this further tonight, however. |
Cc: @nolta |
This is what I get with runtests("arpack")
|
My MKL machine is down at the moment, but the last time i tried the tests passed. Does setting |
It doesn't for me. I distcleaned arpack/suite sparse and made without
|
This did work for me until recently too - so it is certainly a recent issue. We should certainly make it work with USE_BLAS64 too while we fix this. |
I managed to compile arpack with |
@vtjnash Perhaps we need something similar to gfortblas here? Can you comment? |
@loladiro is correct. ifort uses the standard g77/f2c calling convention rather than the more c-like calling convention that gfortran decided to introduce (arbitrarily, IMHO) much later. maybe we should just "fix" our gfortran compiler on mac by passing "-f2c", and delete the gfortblas wrapper. note however, that this would require changing many of the function signatures in base. |
Maybe push this issue to 0.3 then and try to implement #2167? |
Passing the f2c option seems like the right thing to do. I think we should do it in 0.2 since mkl is important for performance for a lot of people. |
fcall would be nice but that is certainly for the next release. |
What @loladiro and I am trying to say is that those two issues aren't separable. We can't reasonably/easily switch to |
For 0.2, I think the message needs to be that we don't support MKL (it's mentioned in the ifort manual that you can't link gfortran and ifort, and julia = gfortran for all intents and purposes) |
Ok. Got it. I thought that they were two separate things - as in one was a quick fix and the other one longer term. |
This need |
MKL has a version compiled with gfortran, but we are unable to successfully link to it and use it. |
Note that |
This may or may not be helpful: https://software.intel.com/en-us/articles/how-to-resolve-arpack-issues-with-intel-mkl-110-update-3 I hit this same problem while building Julia this last month. |
@ViralBShah fcall would help address this, because we could then build arpack with |
It may be simpler to just build everything with Intel compilers when using MKL. The MKL does ship with gfortran linked libs, but it didn't work the last I tried, which should have fixed this. Did that blog post resolve the issue? |
@vtjnash Does Intel use the f2c calling convention? In that case, yes, |
It is my understanding currently that they are the same. Of course, if you have a license to MKL, you're more likely to have a license to the intel compiler too. |
I am willing to try using Intel compilers if you can tell me how I should do it. |
Theoretically would like things to work using just the environment variables that were tried as part of #6917. We probably need additional rpath flags in order to get LLVM to link properly with Intel compilers. |
Try |
I have built Julia using Intel compilers and am getting this problem when running
After which tests seem to stall and not run anymore. Here's what I used:
|
That looks a little bit like JuliaMath/openlibm#57, but with the difference being intel compiler rather than i486 arch. Openlibm has some problems, apparently. I don't see any signs of |
Hmm. I changed
|
(just checking) was that trying to re-use a previously built libopenspecfun? If so, may need The issue with |
As part of #7547, when compiling everything with intel compilers, everything works fine. I am satisfied enough with this and will close this issue when the icc PR is merged. |
Fixed in #7547. Resolution is to build with Intel compilers when using MKL. |
The arpack test crashes when julia is built with USE_MKL. This needs to be verified again and fixed.
The text was updated successfully, but these errors were encountered: