Skip to content

Conversation

mkrainiuk
Copy link
Contributor

Description
As a follow-up from #846 review this PR adds an initial documentation for CBLAS interface with 64 extended API.

The document style is aligned with https://github.com/Reference-LAPACK/lapack/blob/master/DOCS/lapacke.pdf but for better readability/maintenance on the GitHub it's implemented in Markdown.

The document preview: https://github.com/mkrainiuk/lapack/blob/master/DOCS/CBLAS.md

langou
langou previously approved these changes Jun 21, 2023
@mkrainiuk
Copy link
Contributor Author

Hi @langou, could you please help me to merge this CBLAS doc PR since CBLAS API changes #857 were already merged?

DOCS/CBLAS.md Outdated
and add the prefix `cblas_`. For example, the BLAS routine `DGEMM` becomes `cblas_dgemm`.

CBLAS routines also support `_64` suffix that enables large data arrays support in the LP64 interface library
(default build configuration). This suffix allows to mix LP64 and ILP64 programming models in one application.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammatically, should be "allows mixing" instead of "allows to mix".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for catching it! Fixed.

@langou langou merged commit a1f308a into Reference-LAPACK:master Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants