Problems with BLAS Documentation #1153
bogdan-barbu
started this conversation in
General
Replies: 1 comment
-
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
First of all, since BLAST doesn't document the legacy BLAS but instead refers to the original Fortran 77 definition, I've noticed a few inconsistencies which I am going to be a bit pedantic about:
cblas_chpr
andcblas_zhpr
take aconst float *alpha
andconst double *alpha
parameter, respectively, when these shouldn't be pointers. Given that this header is normative and linked from the official specification, I think this should be fixed. The second header, which I believe was taken directly from an old version of LAPACK, can be found here and actually fixes the previous problem but also has acblas_xerbla
which I believe is a vendor-specific error handler. So long as this is removed, this could probably just replace the header linked from the spec; maybe theextern "C"
should also be removed given the spec doesn't care about C++ and that's up to the implementor to add if they want to. Apart from that, the only differences between the two seem to be in terms of whitespace. The last header is the one from the current LAPACK, and has all sorts of LAPACK-specific extensions. At any rate, none of these header seem to have the routines mentioned above, at point 1.I would also suggest, if there aren't any copyright issues, to maybe post the level 1/2/3 documents on the Web site. The BLAST standard only references them but anyone who wants to implement legacy BLAS with the guidance from Annex B has to go search for this stuff on the Internet on their own. It would be nice to have everything in one place.
Beta Was this translation helpful? Give feedback.
All reactions