Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Conversation

@Zha0q1
Copy link
Contributor

@Zha0q1 Zha0q1 commented Sep 17, 2020

In mxnet we wrap lapack functions in c_lapack_api.h so that we hide the differences in the underlying blas/lapack libraries such as mkl, openblas, atlas, and accelerate.

For mkl we use/wrap the LAPACKE (https://www.netlib.org/lapack/lapacke.html) c interfaces which are the cleanest. For the rest of the libraries we wrap the old CLAPACK interfaces.

While atlas and accelerate don't provide the LAPACKE interfaces, openblas does have it. This pr makes mxnet also use the LAPACKE apis when openblas is chosen.

This change will make ilp64 blas/lapack support easier as now we have the same wrapping logic or both mkl and openblas. Support for ilp64 mkl #19067 will automatically mean support for ilp64 openblas.

@mxnet-bot
Copy link

Hey @Zha0q1 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [edge, windows-cpu, sanity, website, clang, windows-gpu, centos-gpu, centos-cpu, unix-gpu, unix-cpu, miscellaneous]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@Zha0q1 Zha0q1 changed the title Make OpenBLAS use the LAPACKE apis [poc] Make OpenBLAS use the LAPACKE apis Sep 17, 2020
@Zha0q1 Zha0q1 closed this Sep 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants