-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Consider not to use OpenBLAS, which is not well maintained #3281
Comments
If we choose mklml + mkldnn, yeah, we can remove this dependency from paddle. |
PR #3244 remove the global linker and exe from mkldnn iomp. So we can use mklml + mkldnn instead of openblas. @wangkuiyi @gangliao |
@wangkuiyi @luotao1 I found it's impossible to achieve this goal now. Because our Android version heavily depends on OpenBLAS |
In the X86 environment, there are many BLAS can be selected, such as MKL, OpenBlas, Eigen, and so on. |
I'm closing this since we are implementing support of MKL, feel free to reopen. |
Now, there are 816 Wunused-but-set-variable warning
Wunused-variable warning
|
Besides, there are 18
|
Not helpful, real warning is in first line, only 2nd visible in your post. We have GCC too to see same warnings. |
I grep-ed our TeamCity log -- all 788
-Wunused-but-set-variable
warnings come from OpenBLAS, as pasted below to give you a feeling how many is 788.There are 154 open issues in OpenBLAS's Github repo: https://github.com/xianyi/OpenBLAS/issues?page=7&q=is%3Aissue+is%3Aopen. The earliest was opened in May 2012.
The current Travis-CI status of OpenBLAS is "failed".
The author's homepage http://xianyi.github.io/ shows that he is running his own business in Beijing, PerfXLab, whose one business is to consult OpenBLAS, but obviously this repo is not under industrial quality maintainence.
Do we have to depend on OpenBLAS? Can we remove it from our dependency list?
The text was updated successfully, but these errors were encountered: