search

ralna / spral

Sparse Parallel Robust Algorithms Library
https://ralna.github.io/spral/
Other
104 stars 27 forks source link

[CI] remove Intel until we can link MKL #150

Closed jfowkes closed 11 months ago

jfowkes commented 11 months ago

Resolves #148

jfowkes commented 11 months ago

@amontoison we need to link with MKL not OpenBLAS for the Intel compilers! Unless you can find a way to make this work on GitHub Actions I will be forced to remove the Intel tests in this PR.

amontoison commented 11 months ago

@jfowkes Is it not possible to just link with -lblas and -llapack?

OpenBLAS is great but I forgot that we download a precompiled version, which is probably compiled with gnu compilers. We can't easily mix libraries compiled with gnu / intel compilers.

jfowkes commented 11 months ago

@amontoison no sorry that doesn't work, only Intel Compilers + Intel MKL works successfully (again the reference BLAS/LAPACK libraries are compiled for GNU). If there is no way to link in MKL then I will comment out the Intel tests.

amontoison commented 11 months ago

@jfowkes I propose to not spend more time on the old build system. We know that GNU compilers are working for Linux and Mac. Meson is needed for supporting more compilers or platforms now.

jfowkes commented 11 months ago

Agreed @amontoison, we also know that the Intel compilers are working for Linux with MKL (tested locally on irons). Let's switch over to Meson and get rid of this ancient autotools!!