Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Add support for netlib reference cblas in the CMake build system. #5333
+5
−1
Conversation
de-vri-es
commented
Mar 12, 2017
|
Ping? There should be no risk of breaking anything that currently works, the user has to specify the CBLAS provider manually. |
de-vri-es
commented
May 7, 2017
|
Could this be reviewed? It's pretty low risk. |
de-vri-es
commented
May 30, 2017
|
There is absolutely no risk of breaking anything that currently works, this only adds an extra blas provider. I'd think it would be easy to review and merge this. |
de-vri-es
commented
Jul 11, 2017
|
Seriously, this is an extremely low risk change. Can somebody please review this? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
de-vri-es commentedFeb 28, 2017
The reference
cblasfrom http://www.netlib.org/lapack/ seems to work fine with caffe, and is widely available in a large number of distros, so it makes sense to support it. This PR adds it as supported BLAS provider for the cmake build.The reference
cblasships with acblas-config.cmakefile, so no additional cmake module is needed.