Skip to content

Create DOCS/CBLAS.md #853

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

Merged
merged 2 commits into from
Jul 4, 2023
Merged

Create DOCS/CBLAS.md #853

merged 2 commits into from
Jul 4, 2023

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