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

Arm64 Row-Major d8x6 #650

Closed
wants to merge 2 commits into from
Closed

Arm64 Row-Major d8x6 #650

wants to merge 2 commits into from

Conversation

xrq-phys
Copy link
Collaborator

@xrq-phys xrq-phys commented Aug 1, 2022

Basically it's a templated re-implementation of current column-major d6x8's transpose.

Shall I test out gemmtrsm?

Basically it's a templated re-implementation of current 6x8's transpose.
@fgvanzee
Copy link
Member

fgvanzee commented Aug 2, 2022

Thanks @xrq-phys! (@jdiamondGitHub will definitely be interested in this.)

Shall I test out gemmtrsm?

You are welcome to give it a try! It would have a great benefit for trsm users. 🙂

Contact me on Discord if you have any questions regarding the semantics of gemmtrsm. It can be confusing the first time, especially the _u (upper triangular) instance since it moves backwards (bottom-right to top-left).

@jdiamondGitHub
Copy link
Member

Yes, thank you so much @xrq-phys. :) We are in a crunch at the moment, but we'll try to give it a run within the next week.

@fgvanzee
Copy link
Member

fgvanzee commented Aug 5, 2022

(Apologies @xrq-phys and @figual for the earlier (deleted) message: I got my PRs mixed up.)

@fgvanzee
Copy link
Member

Closing this PR. With the merging of #655, it is now moot. Thanks @xrq-phys!

@fgvanzee fgvanzee closed this Aug 30, 2022
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