openblas-src icon indicating copy to clipboard operation
openblas-src copied to clipboard

Published 20 hours ago • verified publisher icon blas-lapack-rs

Add support for the Intel compiler

Open termoshtt opened this issue 8 years ago • 7 comments

I try to compile this crate with intel-compiler (ifort) and succeed by rewriting dylib specification in build.rs. https://github.com/termoshtt/openblas-src/pull/1/files Do you have any plan to support intel compiler? If you are interest in, I will create PR to introduce intel feature to switch gfortran/ifort.

termoshtt avatar Apr 10 '17 06:04 termoshtt

Thank you for the issue! Yes, it certainly would be great to support different configurations.

I suppose OpenBLAS detected by itself that gfortran was not available and switched to ifort, right? If it’s the case, the way to proceed is to fetch the compiled utilized from OpenBLAS’s log files or something and then link to the right libraries in the build script. I was looking it to this some time ago and was able to do this, but then I dropped that code as I had only one alternative, gfortran, and was not able to test what would happen with other potential compilers.

IvanUkhov avatar Apr 10 '17 12:04 IvanUkhov

Have a look at the code that I’m removing in this commit.

IvanUkhov avatar Apr 11 '17 03:04 IvanUkhov

Thanks link!

the way to proceed is to fetch the compiled utilized from OpenBLAS’s log files or something and then link to the right libraries in the build script.

It seems to be more complicated than I intended, but it is feasible as you did. My concern is that it is hard to test on CI using ifort since there is no option to use Intel compiler on CI as I sought. If I create gfortran/ifort switch, it will be tested only on my environment. Does this crate permit this?

termoshtt avatar Apr 11 '17 04:04 termoshtt

Yes, I think it’s worth it. There will be at least one confirmation that it’s working as intended.

IvanUkhov avatar Apr 11 '17 06:04 IvanUkhov

Ok, I will create it. Thanks!

termoshtt avatar Apr 11 '17 06:04 termoshtt

@termoshtt, is this still relevant?

IvanUkhov avatar Feb 20 '21 06:02 IvanUkhov

is this still relevant?

Thanks for notification. The above link is out-of-date completely. But Intel compiler support is still a valid issue.

  • Since the OpenBLAS build system must support icc/ifort, we can use it through openblas-build.
  • A remaining problem is how to test it on CI.
    • I expect we can use the oneAPI container https://hub.docker.com/r/intel/oneapi-basekit for this purpose, though we may need some struggle to setup it

termoshtt avatar Mar 27 '21 08:03 termoshtt