Skip to content
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

implement oneMKL row-major -> cublas mapping #588

Open
JackAKirk opened this issue Oct 10, 2024 · 0 comments
Open

implement oneMKL row-major -> cublas mapping #588

JackAKirk opened this issue Oct 10, 2024 · 0 comments
Labels
BLAS domain BLAS domain issue/request

Comments

@JackAKirk
Copy link
Contributor

JackAKirk commented Oct 10, 2024

After #586 the cublas -> oneMKL mappings are almost as complete as they can be according to the current oneMKL spec; although note that there are cuda cases that aren't exposed via oneMKL, see e.g. #562. Also some low precision types are not implemented yet: #506

However the oneMKL -> cublas mapping is not complete since row_major cases are not implemented.

cublas uses col_major, but takes a transpose bool. oneMKL also exposes a row_major option. rocblas is the same situation, and this patch:

#428

mapped oneMKL row_major to rocblas implementation via clever usage of transposition.

The cublas backend could do the same, probably via mainly just copy pasting the row_major blas implementations from the rocblas backend.

@JackAKirk JackAKirk changed the title impl oneMKL row-major -> cublas mapping implement oneMKL row-major -> cublas mapping Oct 10, 2024
@Rbiessy Rbiessy added the BLAS domain BLAS domain issue/request label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLAS domain BLAS domain issue/request
Projects
None yet
Development

No branches or pull requests

2 participants