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

TARGET=GENERIC produces errors with TRMM() #2257

Closed
BarrySmith opened this issue Sep 13, 2019 · 1 comment · Fixed by #2261
Closed

TARGET=GENERIC produces errors with TRMM() #2257

BarrySmith opened this issue Sep 13, 2019 · 1 comment · Fixed by #2261

Comments

@BarrySmith
Copy link

BarrySmith commented Sep 13, 2019

Building with

make TARGET=GENERIC MAKE_NB_JOBS=13 USE_OPENMP=0 USE_THREAD=0 NO_EXPRECISION=1

produces errors with TRMM(). This happens for both Gnu and Apple CLANG compilers on both Linux and Apple. Both v0.3.6 and v0.3.7. Same error also with USE_THREAD=1 (haven't tested USE_OPENMP=1).

`OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat3 < ./sblat3.dat

OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./dblat2 < ./dblat2.dat

TESTS OF THE REAL LEVEL 3 BLAS

THE FOLLOWING PARAMETER VALUES WILL BE USED:

FOR N 0 1 2 3 7 31

FOR ALPHA 0.0 1.0 0.7

FOR BETA 0.0 1.0 1.3

ROUTINES PASS COMPUTATIONAL TESTS IF TEST RATIO IS LESS THAN 16.00

RELATIVE MACHINE PRECISION IS TAKEN TO BE 1.2E-07

SGEMM PASSED THE TESTS OF ERROR-EXITS

SGEMM PASSED THE COMPUTATIONAL TESTS ( 17496 CALLS)

SSYMM PASSED THE TESTS OF ERROR-EXITS

SSYMM PASSED THE COMPUTATIONAL TESTS ( 1296 CALLS)

STRMM PASSED THE TESTS OF ERROR-EXITS

******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******

       EXPECTED RESULT   COMPUTED RESULT

   1      0.186813          0.373626 

******* STRMM FAILED ON CALL NUMBER:

506: STRMM ('L','U','N','U',  1,  1, 1.0, A,  2, B,  2)        `

Similar errors for all the variants of Xtrmm()

Error does not occur if TARGET=GENERIC is not used on the same systems.

@martin-frbg
Copy link
Collaborator

This was apparently caused by me with PR #2051

martin-frbg added a commit that referenced this issue Sep 17, 2019
* Revert #2051 and add a better fix for TARGET=generic with DYNAMIC_ARCH
fixes #2257 without breaking #2048 again
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 a pull request may close this issue.

2 participants