-
Notifications
You must be signed in to change notification settings - Fork 56
Add openblas with flang #1377
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
Add openblas with flang #1377
Conversation
Doing this for scipy. |
The idea being used here is this (OpenMathLib/OpenBLAS#3640 (comment)) |
Though the above recipe passes I have quite some doubts (or improvements that can be done here)
I haven't used any right now but I guess we would want to focus on some (pyodide uses
I had initially started with Would be great if y'all can guide me here. |
cp libopenblas.a $PREFIX/lib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might not need to do this
mkdir -p $PREFIX/lib
cp libopenblas.a $PREFIX/lib
Cause the file we are interested is libopenblas_riscv64_generic-r0.3.28.a
I suppose (obviously the name can be changed LIBNAMESUFFIX
and we can set it to whatever we want)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some context we use RISCV64_GENERIC
as the target as mentioned by one of OpenBlas' maintainers. Pyodide also uses the same.
For R, I think we would want libopenblas.so so that users can replace libRblas.so with other BLAS libs. But this is optional. For scipy, it might be better to only have libopenblas.a so that it doesn't need to be added as a run dependency. It would be nice to have both options, if possible. |
If |
For lapack I believe we need the shared library, but there is no harm in also having the static one
Whatever is needed for the downstream projects |
No description provided.