-
Notifications
You must be signed in to change notification settings - Fork 480
Implement xGEMMTR and cblas_xGEMMTR #887
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
langou
merged 23 commits into
Reference-LAPACK:master
from
grisuthedragon:implement-gemmt
Jun 28, 2024
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
19b0016
Add xGEMMT and their test cases
grisuthedragon 5daea48
CBLAS routines for xGEMMT added
grisuthedragon 630fb5b
Tests for cblas_cgemmt
grisuthedragon b25cf2c
Update documentation of xGEMMT
grisuthedragon fb5325d
Fix implicit variable
grisuthedragon 2f80551
Fix further implicit variables
grisuthedragon 05d01da
Fix missing comma
grisuthedragon 6f66c83
Remove useless variable
grisuthedragon 785d734
Fix wrong write
grisuthedragon 6173b6e
Fix another variable
grisuthedragon 327869d
Rename GEMMT to GEMMTR in BLAS/
grisuthedragon 81b3767
Change xGEMMT to xGEMMTR in CBLAS/
grisuthedragon cb81e00
Adjust BLAS test for routine names with at most 7 characters
grisuthedragon f9ea71e
Fix missing arguments in tests
grisuthedragon 60d0e76
Adding cblas_zgemmtr test
grisuthedragon b721a55
Working CBLAS_ZGEMMTR Test
grisuthedragon adaf724
Update comments
grisuthedragon b681b1e
Add cblas_dgemmtr test
grisuthedragon 63d2b3a
add cblas_sgemmtr tests
grisuthedragon 8571780
Working error tests on cblas_cgemmtr
grisuthedragon 34adaba
Add tests for cblas_s/d/zgemmtr
grisuthedragon 0e37c5c
Fix CMake Build
grisuthedragon c57c156
Add gemmtr group to Doxygen
grisuthedragon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add tests for cblas_s/d/zgemmtr
- Loading branch information
commit 34adaba0e6829c3cc43cda4aba6e78c44ac93b8b
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the very late reply, but is this part intentional - setting RowMajorStrg TRUE but then calling gemmtr with CBlasColMajor,... ? Seems all c_?3chke.c have these three tests where the Strg flag for the xerbla truth value swapping does not match the actual layout used (with varied M,K, LDA)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On the first view , it seems to be a mistake, but not on the
RowMajorStrg = TRUE;but on the subsequentcblas_dgemmtrcall. Since the row majors are checked before.