platform/upstream/lapack.git
9 years agoadding end bracket...now looks better
julie [Sun, 15 Nov 2015 18:00:34 +0000 (18:00 +0000)]
adding end bracket...now looks better

9 years agoremoving extra bracket
julie [Sun, 15 Nov 2015 17:56:10 +0000 (17:56 +0000)]
removing extra bracket

9 years agoAdding potrf2 and getrf2 to CMakefile
julie [Sun, 15 Nov 2015 17:43:25 +0000 (17:43 +0000)]
Adding potrf2 and getrf2 to CMakefile

9 years agointegration of xGETRF2
langou [Sun, 15 Nov 2015 17:15:57 +0000 (17:15 +0000)]
integration of xGETRF2

9 years agointegration of xPOTRF2
langou [Sun, 15 Nov 2015 16:49:46 +0000 (16:49 +0000)]
integration of xPOTRF2

9 years agoAdd the missing CONJG ...
julie [Sun, 15 Nov 2015 02:50:39 +0000 (02:50 +0000)]
Add the missing CONJG ...

TESTINGS are all good now!

9 years agoAdding CGESVJ/ZGESVJ and CGEJSV/ZGESVJ to the testing suite
julie [Sun, 15 Nov 2015 02:19:18 +0000 (02:19 +0000)]
Adding CGESVJ/ZGESVJ and CGEJSV/ZGESVJ to the testing suite

Note: TEST 15 and 19 (xBDT01  - | A - U diag(S) VT | / ( |A| max(M,N) ulp )  are not passing the threshold
      TEST 15 and 19 are commented until we find the fix

9 years agoFix cast...
julie [Sat, 14 Nov 2015 06:40:57 +0000 (06:40 +0000)]
Fix cast...

9 years agoAdding *esvdx and *bdsvdx to LAPACKE
julie [Sat, 14 Nov 2015 06:30:55 +0000 (06:30 +0000)]
Adding *esvdx and *bdsvdx to LAPACKE

*** NEED REVIEW ***

9 years agoSee: http://icl.utk.edu/lapack-forum/viewtopic.php?t=4830
langou [Fri, 13 Nov 2015 18:03:21 +0000 (18:03 +0000)]
See: icl.utk.edu/lapack-forum/viewtopic.php?t=4830

The variables NDIV, NITER and NFAIL are INOUT.  These variables have to be
initialized (e.g., in LASQ2) before callin LASQ3.  LASQ3 increments these
counters.  These counters do not play a role in the algorithm; they just give
statistics.) Thanks to Tracey Brendan for the report.

9 years agoAdding *gesvj and *gejsv to LAPACKE
julie [Fri, 13 Nov 2015 07:41:33 +0000 (07:41 +0000)]
Adding *gesvj and *gejsv to LAPACKE

9 years agoUpdate Doxyfile... one more
julie [Thu, 12 Nov 2015 15:15:07 +0000 (15:15 +0000)]
Update Doxyfile... one more

9 years agoUpdate Doxyfile
julie [Thu, 12 Nov 2015 15:09:31 +0000 (15:09 +0000)]
Update Doxyfile

9 years ago Rook with 2 o , not 3...
julie [Wed, 11 Nov 2015 06:28:29 +0000 (06:28 +0000)]
 Rook with 2 o , not 3...

9 years agoCommitting patch submitted by Christopher Conrad on Sept 25th 2015
julie [Wed, 11 Nov 2015 06:20:49 +0000 (06:20 +0000)]
Committing patch submitted by Christopher Conrad on Sept 25th 2015

From Christopher:
I noticed the following errors in the documentation and attached
patches for all of them:
- Sometimes the function name in the docstring \brief does not match the
actual function name (ex: xSYGV*).
- According to the documentation, xLAHRD is obsolete but not deprecated.
- The parameter N in xLA(R)SCL2 is not the number of columns of D.

Also, I updated the Doxyfile (doxygen -u DOCS/Doxyfile) and removed
the missing DOT_FONTNAME to get rid of the Doxygen warnings.

9 years agoAdding xlapmt to LAPACKE ( continue ) .. forgot to save the include file
julie [Wed, 11 Nov 2015 05:52:56 +0000 (05:52 +0000)]
Adding xlapmt to LAPACKE ( continue ) .. forgot to save the include file

9 years agoAdding xlapmt to LAPACKE
julie [Wed, 11 Nov 2015 05:40:19 +0000 (05:40 +0000)]
Adding xlapmt to LAPACKE

9 years agoFix cast for lwork in complex [cz]uncsd2by1 routines
julie [Wed, 11 Nov 2015 05:13:21 +0000 (05:13 +0000)]
Fix cast for lwork in complex [cz]uncsd2by1 routines

9 years agoIncorporating DBDSVDX tests in dchkbd
julie [Wed, 11 Nov 2015 03:35:02 +0000 (03:35 +0000)]
Incorporating DBDSVDX tests in dchkbd

Disabling temporarily some [SD]BDSVDX tests - those generate some numerical errors
See email from Osni on Nov 10th 2015

9 years agoAdding routines to Makefile / CMakefile.txt
julie [Mon, 9 Nov 2015 00:13:42 +0000 (00:13 +0000)]
Adding routines to Makefile / CMakefile.txt

9 years agoCommitting Zlatko Drmac Contribution
julie [Mon, 9 Nov 2015 00:10:27 +0000 (00:10 +0000)]
Committing Zlatko Drmac Contribution

9 years agoFirst commit for Zlatko Drmac Contribution - Fixing z precisions issues + modif sent...
julie [Sun, 8 Nov 2015 23:42:08 +0000 (23:42 +0000)]
First commit for Zlatko Drmac Contribution - Fixing z precisions issues + modif sent by Zlatko

9 years agoFirst commit for Zlatko Drmac Contribution
julie [Sun, 8 Nov 2015 23:24:14 +0000 (23:24 +0000)]
First commit for Zlatko Drmac Contribution

Contains only source code with Doxygen format and Makefile modifications for
CGESVJ (Jacobi SVD) with its dependencies: CGSVJ0 and CGSVJ1.

TO DO:
  - Include Testing
  - LAPACKE interface

9 years agoChange DBLE cast by REAL - confirmed with Osni
julie [Fri, 6 Nov 2015 01:16:19 +0000 (01:16 +0000)]
Change DBLE cast by REAL - confirmed with Osni

9 years agoThanks to Lawrence Mulholland (NAG) for reporting the problem. While the array
langou [Wed, 4 Nov 2015 19:54:31 +0000 (19:54 +0000)]
Thanks to Lawrence Mulholland (NAG) for reporting the problem.  While the array
E should really be of size N-1, xSYCONV() is not ready for this and so, in the
comment at the header level, we now require E to be of size N. Also xSYTRS2()
is using E(N) = ZERO, so if we make E of size N-1 in xSYCONV(), then we need to
see the impact on xSYTRS2().  This is long term cleaning, so for now, we revert
back to requesting an array E of size N. Thanks Lawrence for noticing this.

9 years agoCommit svdx files sent by Osni on Oct 31st 2015
julie [Wed, 4 Nov 2015 06:22:56 +0000 (06:22 +0000)]
Commit  svdx files sent by Osni on Oct 31st 2015
Updated Makefile
Updated file format (Oxygen)
Small Fix: sdrvbd / cdrvbd has a lot of double / double complex in comments

From Osni:
Files for LAPACK/SRC (all new, additions to Makefile are needed):

 - dbdsvdx.f, sbdsvdx.f: full or partial (subset) SVD of a bidiagonal matrix through an associated eigenvalue problem
 - dgesvdx.f, sgesvdx.f, zgesvdx.f, cgesvdx.f: full or partial (subset) SVD of a general matrix by invoking bdsvdx

Files for LAPACK/TESTING/EIG :
- dchkbd.f, schkbd.f: added tests 20-34 for bdsvdx
- dbdt04.f, sbdt04.f (new): needed for tests 25 and 30 in chkbd
- dlahd2.f, slahd2.f: added information about tests 20-34 in chkbd
- derrbd.f, serrbd.f: added tests for the values of INFO returned by bdsvdx
- ddrvbd.f, sdrvbd.f, zdrvbd.f, cdrvbd.f: added tests 23-35 (real case) and 15-27 (complex case) for gesvdx.
- dbdt05.f, sbdt05.f, zbdt05.f, cbdt05.f (new): needed for tests 30,33 (real case) and 22,25 (complex case) for gesvdx
- derred.f, serred.f, zerred.f, cerred.f: added tests for the values of INFO returned by gesvdx

Current Issues:
 - 16 real tests do not pass the threshold ( ssvd.out) - but seems to go through in debug
 - serrbd.f has some DOUBLE PRECISION calculation inside - Is that ok or shall we change it to REAL?
Line 113-119
*     Set the variables to innocuous values.
*
      DO 20 J = 1, NMAX
         DO 10 I = 1, NMAX
            A( I, J ) = 1.D0 / DBLE( I+J )
   10    CONTINUE
   20 CONTINUE

TODO:
     - corresponding LAPACKE routines
     - Test Doxygen format

9 years agoFix some typos..
julie [Wed, 4 Nov 2015 05:34:49 +0000 (05:34 +0000)]
Fix some typos..

9 years agoremove extra tabs - Thank you Matyas A Sustik
julie [Sun, 1 Nov 2015 16:21:47 +0000 (16:21 +0000)]
remove extra tabs - Thank you Matyas A Sustik

9 years agoIn ?syconv, replace the variable name WORK by the variable name E. E is the
langou [Thu, 29 Oct 2015 05:06:14 +0000 (05:06 +0000)]
In ?syconv, replace the variable name WORK by the variable name E.  E is the
standard way to name the supdiagonal/subdiagonal of a symmetric tridiagonal
matrix.  Also, E (previously WORK) is of size N-1, not N. So correct this in
the comment.

9 years agoThe subroutine DSYCONV stores the supdiagonal of the symmetric 1-by-1 or 2-by-2
langou [Thu, 29 Oct 2015 04:58:50 +0000 (04:58 +0000)]
The subroutine DSYCONV stores the supdiagonal of the symmetric 1-by-1 or 2-by-2
block matrix in the array WORK. (Or the subdiagonal, same array.) Before
commit, only the nonzero elements of WORK were initialized to their correct
nonzero values. The zero entries were not initialized and not referenced and
assumed to be zero in subsequent subroutines. This is fine, but I think a
better behavior for the subroutine is to initialize these zero entries to zero.
So the commit is initializing these entries to zero.

Also a (double precision) ZERO in the D routine should have been an (integer)
0.

Also fix some formatting to have C, D, and S consistent.

Note: the indentations, comments of the Z routine is not consistent with C, D and
S. I did not have time to fix this.

9 years agominor: (1) eliminate trailing spaces in comments and (2) comment indicators
langou [Thu, 29 Oct 2015 04:41:20 +0000 (04:41 +0000)]
minor: (1) eliminate trailing spaces in comments and (2) comment indicators
along the left margin of the commented fragment, in order to show explicitly
that the fragment is a comment rather than a code between two comments.
Thanks Dmitry.

9 years agoRemove NaN check from LAPACKE_?laswp and add a comment on why the NaN check is removed.
langou [Wed, 28 Oct 2015 02:35:58 +0000 (02:35 +0000)]
Remove NaN check from LAPACKE_?laswp and add a comment on why the NaN check is removed.
This might not be the best fix but this will do for now.
Current behavior: LAPACKE_?laswp does not check for NaNs.
See: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?t=4827 for more information.
Thanks to Dmitry Baksheev from Intel.

9 years agoApply Christoph Conrads Doxygen patch for xyycsd2by1.f - comment modification only
julie [Tue, 27 Oct 2015 14:10:26 +0000 (14:10 +0000)]
Apply Christoph Conrads Doxygen patch for xyycsd2by1.f - comment modification only

9 years agoCommitting temporary fix for bug 113
julie [Tue, 27 Oct 2015 03:16:53 +0000 (03:16 +0000)]
Committing temporary fix for bug 113
Submitted by Osni Marques on Oct 22nd 2015
we just need to set the variable NOFAIL to .false. in _larrf.f (line 260).
As it is now, dsyevr may enter into an infinite loop in some rare cases.
With NOFAIL=.false. the code will return an error message.

9 years agoFix bug 137 and 138 submitted by Dan Kortschak
julie [Thu, 22 Oct 2015 05:29:34 +0000 (05:29 +0000)]
Fix bug 137 and 138 submitted by Dan Kortschak

Applied patch submitted by Dan Kortschak

9 years agoFix bug 139 submitted by Christoph Conrads on Oct 6th 2015
julie [Thu, 22 Oct 2015 04:39:19 +0000 (04:39 +0000)]
Fix bug 139 submitted by Christoph Conrads on Oct 6th 2015
"according to the LAPACK documentation, xSTEDC guarantees to
return the eigenvalues in ascending order (parameter D on exit). To
this end, it sorts the eigenvalues if necessary.

If there is only a single subproblem of size n, no sorting algorithm is
called (cf. {s,d}tedc.f, line 450). Furthermore, xLAED0
({s,d}tedc.f:400) does not guarantee to return eigenvalues in ascending
order. Thus, xSTEDC may return eigenvalues that are not in ascending
order."

Applied patch provided by Christoph Conrads: "always sorting the eigenvalues"

9 years agobugs in ?GGHD3 : B,LDA-->B,LDB and zeroing of too large a submatrix
lawrence.mulholland [Tue, 13 Oct 2015 12:10:47 +0000 (12:10 +0000)]
bugs in ?GGHD3 : B,LDA-->B,LDB and zeroing of too large a submatrix

9 years agoQuick and dirty bug fix.
langou [Tue, 6 Oct 2015 02:11:30 +0000 (02:11 +0000)]
Quick and dirty bug fix.

Bug report and bug fix from Dan Kortschak.

See:
http://icl.cs.utk.edu/lapack-forum/viewtopic.php?t=4810
https://github.com/xianyi/OpenBLAS/issues/642

This is a quick bug fix, this only checks the triangular part of the trapezoid,
the rectangular part is not checked. Another commit should follow up soon.

9 years agoAdding [un/or]csd2by1 routines to LAPACKE
julie [Mon, 5 Oct 2015 01:26:04 +0000 (01:26 +0000)]
Adding [un/or]csd2by1 routines to LAPACKE

9 years agoAdding xlascl to LAPACKE
julie [Mon, 5 Oct 2015 00:04:38 +0000 (00:04 +0000)]
Adding xlascl to LAPACKE

9 years ago Updating Copyright with current year
julie [Sun, 4 Oct 2015 18:57:19 +0000 (18:57 +0000)]
 Updating Copyright with current year

9 years agoAdding xsytrf_rook, xsytrs_rook, xhetrf_rook and xhetrs_rook to LAPACKE
julie [Fri, 25 Sep 2015 05:15:44 +0000 (05:15 +0000)]
Adding xsytrf_rook, xsytrs_rook, xhetrf_rook and xhetrs_rook to LAPACKE

9 years agoAdding xlahrd to DEPRECATED (problem reported by zerothi)
julie [Sat, 19 Sep 2015 19:22:02 +0000 (19:22 +0000)]
Adding xlahrd to DEPRECATED (problem reported by zerothi)

9 years agoFixing comments in [sd]lasd6
julie [Sat, 19 Sep 2015 16:46:16 +0000 (16:46 +0000)]
Fixing comments in [sd]lasd6

Issue : Parameter IDXQ of function SLASD6 is described as [out], should be described as [in,out].
Reported by dbakshee from INTEL on the forum on Sep 18th 2015
Link: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4799
9 years agoHandle Deprecated routines in LAPACKE Makefile
julie [Wed, 16 Sep 2015 05:12:31 +0000 (05:12 +0000)]
Handle Deprecated routines in LAPACKE Makefile

9 years agoIssues and fixes submitted by Christoph Conrads on Sept 14th by email
julie [Tue, 15 Sep 2015 07:12:59 +0000 (07:12 +0000)]
Issues and fixes submitted by Christoph Conrads on Sept 14th by email
(1st commit)

-> File modified: DOCS/Doxyfile --
- Doxygen documentation
The Doxygen "project version" is still 3.4.2. The attached patch bumps
this to 3.6.0.
-> let's be ready!
- Doxygen documentation
The Doxygen configuration file uses multiple obsolete tags and warnings
are shown right after starting Doxygen. The warnings are easy to see if
you set "QUIET = YES" in DOCS/Doxyfile.
-> set "QUIET = YES"

-> File modified: SRC/*larfgp.f
It says "non-negatibe" in the documentation of xLARFGP.
-> Fixed typo

-> File modified: SRC/*orcsd2by1.f SRC/*uncsd2by1.f
The parameter M in the xORCSD2BY1 is not the number of columns of X and
there is no X12 (parameter P).
-> Fixed

-> File modified: LAPACKE/src/CMakeLists.txt
The C interfaces for the deprecated GSVD routines are still built with
cmake -DBUILD_DEPRECATED=OFF. This causes a linker error with cmake
-DBUILD_TESTING=ON as soon as the Fortran functions are not found in
liblapack.so.
-> Add a if(BUILD_DEPRECATED)

-> File modified: trunk/LAPACKE/include/lapacke.h
The C declaration (lapacke.h:11840-11843) for xGGSVD3 is in a block
of #defines for LAPACK 3.2.X (see lapacke.h:12089,12122).
-> moved them down

9 years agoBug Fix for bug0135
julie [Thu, 10 Sep 2015 07:11:18 +0000 (07:11 +0000)]
Bug Fix for bug0135
Bug reported by kortschak on Sep 10th
Link: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4793
Fix typo substituting O with 0

9 years agoFixing bug 133
julie [Thu, 3 Sep 2015 06:07:59 +0000 (06:07 +0000)]
Fixing bug 133

Bug reported by the GONUM team member: btracey on LAPACK forum
Link: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4771
Description:  the number of columns of A depends on the Side, so the test should compare against m or n.

Fix by Julie.

9 years agoMinor edits in the BLAS header to better describe the functionality.
langou [Thu, 3 Sep 2015 03:05:54 +0000 (03:05 +0000)]
Minor edits in the BLAS header to better describe the functionality.
See: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?t=4786

9 years agoCommitting Nico's change about build system improvements for LAPACK
julie [Wed, 2 Sep 2015 05:32:19 +0000 (05:32 +0000)]
Committing Nico's change about build system improvements for LAPACK

"One of the improvements with the patches is that SOVERSION and VERSION are now properly set and the proper symlinks are created:
```
liblapack.so -> liblapack.so.3
liblapack.so.3 -> liblapack.so.3.5.0
liblapack.so.3.5.0
```
Since BLAS is shipped with LAPACK and no separate version number is given for BLAS, I applied the same there, too."

Tested on Julie's Mac

TO DO: To test under Windows
====

Updated OSX RPATH settings
In response to CMake 3.0 generating warnings regarding policy CMP0042,
the OSX RPATH settings have been updated per recommendations found
in the CMake Wiki:
  http://www.cmake.org/Wiki/CMake_RPATH_handling#Mac_OS_X_and_the_RPATH

9 years agoA is input/output for the subroutines ?sytrs2.f
langou [Wed, 26 Aug 2015 04:47:36 +0000 (04:47 +0000)]
A is input/output for the subroutines ?sytrs2.f

9 years agoCorrecting Bug 129
julie [Wed, 26 Aug 2015 04:46:55 +0000 (04:46 +0000)]
Correcting Bug 129
See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4747

 Modify LAPACK_?syconv interface  - Work is used to return values

 (1) Remove the local work variable and its allocation and deallocation
 (2) Add work in LAPACK_?syconv interface
 (3) Update lapacke.h

9 years agocorrecting bug 128
langou [Wed, 26 Aug 2015 04:42:47 +0000 (04:42 +0000)]
correcting bug 128
See: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?t=4746

When LAPACK_NAN_CHECK is enabled, in input of xLASET, we now allow if the matrix A contains NaN.

Since the goal of xLASET is to set values in A, we consider it OK if A contains NaN in input.

In other words, the rationale is that we do not consider A has an input/output. We consider A as an output only.

Note: We allow neither alpha nor beta to be a NaN.

(So we do check for NaN in alpha and in beta and return error if NaN.)

So this commit
(1) removes the NaN check on matrix A in LAPACKE wrappers: lapacke_?laset.c
(2) declares that A is OUTPUT in LAPACK subroutines: ?laset.f
(3) adds a thanks to Victor Kostin from Intel

9 years agoPropagate comment added in rev 1568 to make.inc with gfortran compiler
julie [Wed, 26 Aug 2015 01:43:34 +0000 (01:43 +0000)]
Propagate comment added in rev 1568 to make.inc with gfortran compiler

9 years agochanged variable names from LAM_MAX to SING_MAX and from LAM_MIN to SING_MIN to refle...
igor175 [Wed, 19 Aug 2015 21:17:27 +0000 (21:17 +0000)]
changed variable names from LAM_MAX to SING_MAX and from LAM_MIN to SING_MIN to reflect the chnage in the algorithm for TEST4 for rook pivotig routines, since we now use singular values instead of eigenvalues in TEST4 to compute the norm for 2-by-2 marices in TEST 4. Also corrected the bug in for Hermitian case in TEST4, where during the constraction of Hermitian 2-by-2 matrices the operation of complex conjugation was missing. Changes made to: cchkhe_rook.f cchksy_rook.f dchksy_rook.f schksy_rook.f zchkhe_rook.f zchksy_rook.f

9 years agoMoved deprecated SGVD routines into DEPRECATED directory.
philippe.theveny [Mon, 17 Aug 2015 22:58:57 +0000 (22:58 +0000)]
Moved deprecated SGVD routines into DEPRECATED directory.
Added a BUILD_DEPRECATED option for make and cmake, which build
deprecated routines into the LAPACK library (OFF by default).
This completes r1563.

TODO: document the BUILD_DEPRECATED option (i.e. update documentation).

9 years agoRemoved unused and obsolete statement functions.
philippe.theveny [Mon, 17 Aug 2015 21:37:47 +0000 (21:37 +0000)]
Removed unused and obsolete statement functions.

9 years agoAdded LAPACKE wrappers for GSVD (complements r1572).
philippe.theveny [Mon, 17 Aug 2015 21:17:37 +0000 (21:17 +0000)]
Added LAPACKE wrappers for GSVD (complements r1572).

9 years agoAdded BLAS3 routines for generalised SVD.
philippe.theveny [Fri, 14 Aug 2015 22:54:14 +0000 (22:54 +0000)]
Added BLAS3 routines for generalised SVD.

TODO: LAPACKE wrappers.

9 years agoFixed BUG number 123, that is testing of 'rook' routines, specifically TEST 4 for...
igor175 [Tue, 11 Aug 2015 22:27:27 +0000 (22:27 +0000)]
Fixed BUG number 123, that is testing of 'rook' routines, specifically TEST 4 for UPLO='U' case that test bounds of block diagonal matrices. TEST 4 wasfailing for UPLO='U', since 2-by-2 was passed incorrectly to the -EVX routine.  Also, during the fix the  TEST 4 for both UPLO='U' and 'L' was modified. Instead of computing the ratio of eigenvalues for 2-by-2 matrices, using the -EVX routines, now we compute the ratio of singular values using *GESVD routines. Affected test routines: TESTING/LIN/schksy_rook.f TESTING/LIN/dchksy_rook.f TESTING/LIN/cchksy_rook.f TESTING/LIN/zchksy_rook.f TESTING/LIN/cchkhe_rook.f TESTING/LIN/zchkhe_rook.f

9 years agoRemoved declaration of unused variables.
philippe.theveny [Tue, 11 Aug 2015 21:52:11 +0000 (21:52 +0000)]
Removed declaration of unused variables.

9 years agoReplaced archaic intrinsics AMIN1, DMIN1, MIN0, AMAX1, DMAX1, MAX0 by MIN/MAX.
philippe.theveny [Tue, 11 Aug 2015 20:44:38 +0000 (20:44 +0000)]
Replaced archaic intrinsics AMIN1, DMIN1, MIN0, AMAX1, DMAX1, MAX0 by MIN/MAX.

9 years agoadding a comment in make.inc.example saying the following:
langou [Tue, 11 Aug 2015 16:59:56 +0000 (16:59 +0000)]
adding a comment in make.inc.example saying the following:

#
#  Note: During a regular execution, LAPACK might create NaN and Inf
#  and handle these quantities appropriately. As a consequence, one
#  should not compile LAPACK with flags such as -ffpe-trap=overflow.
#

Julien.

9 years agoFix bugs reported by Vitorrio Zecca on August 10, 2015.
philippe.theveny [Tue, 11 Aug 2015 16:23:45 +0000 (16:23 +0000)]
Fix bugs reported by Vitorrio Zecca on August 10, 2015.

9 years agoFix bug reported by Vittorio Zecca.
philippe.theveny [Tue, 11 Aug 2015 16:03:54 +0000 (16:03 +0000)]
Fix bug reported by Vittorio Zecca.

9 years agoBux fix from Rodney James about implicit typing.
philippe.theveny [Mon, 10 Aug 2015 17:33:47 +0000 (17:33 +0000)]
Bux fix from Rodney James about implicit typing.

Some variables were undeclared, some were mistyped, and others were unused.

9 years agoMove all but one deprecated routines to a new separate directory.
philippe.theveny [Thu, 6 Aug 2015 23:01:56 +0000 (23:01 +0000)]
Move all but one deprecated routines to a new separate directory.
They are compiled and included in the liblapack.a archive when
the MAKEDEPRECATED option in the make.inc file is set.

TODO: Add the corresponding behavior to the cmake files.

NOTE: The xGEQPF routines are deprecated but still used by some
non-deprecated routines. They have not been moved to the DEPRECATED
directory.

9 years agoRemove deprecated xGEQPF from the test suite.
philippe.theveny [Thu, 6 Aug 2015 20:57:14 +0000 (20:57 +0000)]
Remove deprecated xGEQPF from the test suite.

WARNING: xGEQPF is still used in xGGSVP routines.

9 years agoRemove all but one deprecated routines from the test suite.
philippe.theveny [Thu, 6 Aug 2015 17:56:35 +0000 (17:56 +0000)]
Remove all but one deprecated routines from the test suite.

NOTE: The tests are renumbered in xDRVLS and xCHKTZ.
TODO: remove deprecated xGEQPF, when it is replaced by xGEQP3 in xGGSVP.

9 years agomissing EXTERNAL
langou [Wed, 5 Aug 2015 21:24:32 +0000 (21:24 +0000)]
missing EXTERNAL

9 years agoCommit bug reported by Alexander Kobotov (INTEL) on LAPACK Forum
julie [Sat, 18 Jul 2015 23:53:25 +0000 (23:53 +0000)]
Commit bug reported by Alexander Kobotov (INTEL) on LAPACK Forum
Link: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4762&p=11439#p11439
Recent commit r1533 introduced out of bound access which could lead to a crash.
It results scaling of a vector instead of a single number.
Fix is simple:

zhgeqz.f:669
- CALL ZSCAL( ILAST, SIGNBC, H( ILAST, ILAST ), 1 )
+ CALL ZSCAL( 1, SIGNBC, H( ILAST, ILAST ), 1 )

Similar for chgeqz on line 668

9 years agoRevert commit 1474: files ?lansy.f are back as they were in revision 1473.
langou [Tue, 30 Jun 2015 10:27:18 +0000 (10:27 +0000)]
Revert commit 1474: files ?lansy.f are back as they were in revision 1473.

9 years ago(minor) Consistency of comments between the four precisions.
langou [Thu, 25 Jun 2015 13:20:58 +0000 (13:20 +0000)]
(minor) Consistency of comments between the four precisions.

9 years agoAdding error test to each element of S must be positive. - Thank you Elena (Oracle)
julie [Thu, 25 Jun 2015 06:00:59 +0000 (06:00 +0000)]
Adding error test to each element of S must be positive. - Thank you Elena (Oracle)

9 years agoFinish removing unnecessary code - TMP = DBLE(M) - see 1547 - Thank you Elena
julie [Thu, 25 Jun 2015 05:09:58 +0000 (05:09 +0000)]
Finish removing unnecessary code - TMP = DBLE(M) - see 1547 - Thank you Elena

9 years agoFix [CDSZ]QRT17.
langou [Mon, 22 Jun 2015 14:04:46 +0000 (14:04 +0000)]
Fix [CDSZ]QRT17.

This is a routine to test DGELS which solves solves full rank overdetermined ( linear least squares ) or underdetermined (minimum norm) linear systems.

The check is, as follows,

(1) If the system is consistent, (smallresidual,) we check that
|| A^T ( b – A * x ) || / (||A||*||b||*max(M,N,NRHS)*eps)
is small (say less than 30).

(2) If the system is not consistent, (large residual,) we check that
|| A^T ( b – A * x ) || / (||A||*|| b - A*x ||*max(M,N,NRHS)*eps)
is small (say less than 30).

DQRT17 was checking in the not consistent case (large residual) that
|| A^T ( b – A * x ) || / (||A||*||x||*max(M,N,NRHS)*eps)
is small. It is a mistake and we need to change the ||x|| by a ||r||.

This fix is consistent with what the documentation says and makes more sense.

9 years agoTypo in the comments of [S,C,D,Z]LATM1.
langou [Fri, 19 Jun 2015 02:59:06 +0000 (02:59 +0000)]
Typo in the comments of [S,C,D,Z]LATM1.
The matrix D needs to be of size N.

9 years agoThese are two fixes in [S,C,D,Z]-LAGGE.
langou [Fri, 19 Jun 2015 02:56:27 +0000 (02:56 +0000)]
These are two fixes in [S,C,D,Z]-LAGGE.

(1) The routine was writing ZEROs out of bound in some specific cases. This is
corrected. Thanks to Yi Meng (CU Denver) for finding the bug.

(2) Quick exit when KL=KU=0 in this case, the user wants a diagonal matrix,
there is probably no reason to call this routine if one simply wants a diagonal
matrix, but that said we still ought to return a correct answer. A quick exit
makes the trick. The routine was not working otherwise.

9 years agobug in [c,d,s,z]get02.f when calling the norm computation for A
langou [Wed, 17 Jun 2015 14:44:54 +0000 (14:44 +0000)]
bug in [c,d,s,z]get02.f when calling the norm computation for A

9 years agotypos
langou [Tue, 16 Jun 2015 14:34:40 +0000 (14:34 +0000)]
typos

9 years agotypos in these testing functions
langou [Mon, 15 Jun 2015 15:30:10 +0000 (15:30 +0000)]
typos in these testing functions

9 years agoIDIST is an INTEGER, change the comment
langou [Wed, 10 Jun 2015 21:41:42 +0000 (21:41 +0000)]
IDIST is an INTEGER, change the comment

9 years agoremove the COMPLEX*16 TMP from DLAHILB
langou [Wed, 10 Jun 2015 17:30:19 +0000 (17:30 +0000)]
remove the COMPLEX*16 TMP from DLAHILB
this has nothing to do here, probably a bad copy-paste from Z to D

9 years agotypos in the comment
langou [Mon, 8 Jun 2015 14:31:13 +0000 (14:31 +0000)]
typos in the comment

9 years agoThis partially fixes bug 061 reported by Victor Liu.
philippe.theveny [Thu, 14 May 2015 18:50:57 +0000 (18:50 +0000)]
This partially fixes bug 061 reported by Victor Liu.

Some compilers allocate local arrays on the heap when their size is
above a particular threshold.
This leads to wrong results when multiple threads call the same routine.

The bug fix consists in using a larger workspace, as proposed by Victor Liu
(Tue Nov 13, 2012).
Some routines still have large local arrays and cannot be fixed that way
because they have no workspace parameter: xGBTRF, xBPTRF, and xHSEQR.

9 years agoApplying patch provided by Julien Schueller (Phimeca) on March 20th 2015 via Lapack...
julie [Wed, 6 May 2015 04:54:45 +0000 (04:54 +0000)]
Applying patch provided by Julien Schueller (Phimeca) on March 20th 2015 via Lapack mailing list to fix some warnings

9 years agoUpdate version number and release date on rook routines - those routines were part...
julie [Wed, 6 May 2015 04:44:20 +0000 (04:44 +0000)]
Update version number and release date on rook routines - those routines were part of the 3.5.0 release - reported by Elena (Oracle)

9 years agoCorrect typo in variable name - reported by Elena (Oracle)
julie [Wed, 6 May 2015 04:30:24 +0000 (04:30 +0000)]
Correct typo in variable name - reported by Elena (Oracle)

9 years agoPropagate fix to bug111 to lapacke - minimum work space - Issue reported by Andrew...
julie [Wed, 6 May 2015 04:21:45 +0000 (04:21 +0000)]
Propagate fix to bug111 to lapacke - minimum work space - Issue reported by Andrew James on LAPACK mailing list on April 30th 2015

9 years agoBug fix from Osni Marques, Beresford Parlett and Jim Demmel.
langou [Tue, 5 May 2015 01:00:18 +0000 (01:00 +0000)]
Bug fix from Osni Marques, Beresford Parlett and Jim Demmel.
This fixes bugs 032 and 056.
Edits in cstein.f, dstein.f, sstein.f and zstein.f

From Osni on Monday, May 4:

As we discussed in our last conference call, I am attaching a new version of
_STEIN, which fixes bugs 032 and 056.  (The bugs were reported for DSTEIN, but
I have propagated the fix to the other versions.) Also, in the process of
testing the fix for those two bugs with more difficult cases, we stumbled upon
a matrix that led SSTEIN to return NaNs in the eigenvectors ... We fixed that
too and, in summary, these are the changes in _STEIN:

1) The assignment after the GO TO 60 needs to be GPIND = J1 (instead of GPIND =
B1).

2) In 'Normalize and scale the righthand side vector Pb' (after GO TO 100 or GO
TO 120) we have replaced _ASUM with I_AMAX.

9 years agoCorrect info testing for SYSVXX routines
julie [Thu, 23 Apr 2015 03:25:41 +0000 (03:25 +0000)]
Correct info testing for SYSVXX routines

in ?SYSVXX
 INFO = -10 needs to be changed to INFO = -11
The code in this section of the files is about to check and make sure all S
elements are positive. And S is argument 11 not 10.

in ?ERRVXX
 Add info testing for input parameter 10 and 11

9 years agoProblem reported by Elena (Oracle)
julie [Wed, 1 Apr 2015 14:56:02 +0000 (14:56 +0000)]
Problem reported by Elena (Oracle)

Some arguments return the wrong Info number
For example LDX is not the 14th argument, but 15th.

Correct corresponding testing routines

9 years ago**** Correct bug 125 ****
langou [Wed, 25 Mar 2015 22:45:19 +0000 (22:45 +0000)]
**** Correct bug 125 ****

Change the maximum number of QR iterations (ITMAX) for xLAHQR from ITMAX=30 to
ITMAX = 30 * MAX( 10, NH ).

Bug reported by Yoshihide Okimi on July 16, 2014
https://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4566
and, independently, by Marco Caliari (Università di Verona) on February 11,
2015.

Involved in bug fix: Yoshihide Okimi, Marco Caliari (Università di Verona),
Meiyue Shao (LBL), Julien Langou, and Daniel Kressner (EPFL).

** Bug **

xLAHQR v3.5 does not converge on some matrices while xLAHQR v2 converges

** Note **

xLAHQR v2 is available through ARPACK so xLAHQR v2 is still in "wide" use

** Description **

xLAQR0 v3.5 and xLAQR4 v3.5 have ITMAX set to:
    ITMAX = MAX( 30, 2*KEXSH )*MAX( 10, ( IHI-ILO+1 ) )

xLAHQR v2 has ITMAX set to:
    ITMAX = 30 * NH

xLAHQR v3.5 has ITMAX set to:
    ITMAX = 30

Yoshihide Okimi reports that, in his application, (1) xLAHQR v3.5 does not
converge, (2) xLAHQR v2 converges, (3) changing ITMAX to 30*NH in xLAHQR v3.5
enables convergence.

Marco Caliari reports that, in his application, using xLAHQR v2.0 converges,
while xLAHQR v3.5 does not converge.

** Bug fix **

Set ITMAX in xLAHQR to
    ITMAX = 30 * MAX( 10, NH )

Email from Meiyue: I feel that setting ITMAX proportional to NH makes sense.
Perhaps we can change ITMAX to ITMAX*MAX( 10, NH ) so that it is also safe for
ill-conditioned tiny matrices.

9 years ago**** Fix bug 124 ****
langou [Wed, 25 Mar 2015 17:28:33 +0000 (17:28 +0000)]
**** Fix bug 124 ****
**** (I also fixed the documentation of the six subroutines ****

Bug reported by Victor_K on the forum.

See forum http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4643

Bug fix by Julien Langou. Involved as well Nick Higham, Sven Hammarling and Craig Lucas.

** Bug **

xPSTF2 and xPSTRF returns NaNs when the matrix in input has N negative diagonal entries.

** Note **

A matrix with N negative diagonal entries is a highly non valid input, but, yet no reason to create some NaNs, better to have a cleaner solution.

** Description **

xPSTF2 and xPSTRF first scan the diagonal of the matrix to find the greatest element on the diagonal.

During this pass on the diagonal, xPSTF2/xPSTRF checks as well for ZERO or NAN on the diagonal and aborts if it finds a ZERO or a NAN on the diagonal.

*
*     Compute stopping value
*
      PVT = 1
      AJJ = A( PVT, PVT )
      DO I = 2, N
         IF( A( I, I ).GT.AJJ ) THEN
            PVT = I
            AJJ = A( PVT, PVT )
         END IF
      END DO
      IF( AJJ.EQ.ZERO.OR.DISNAN( AJJ ) ) THEN
         RANK = 0
         INFO = 1
         GO TO 170
      END IF

Since the input matrix is filled with a negative diagonal (and no ZERO and no NAN), xPSTF2/xPSTRF finds a maximum element (which is the largest negative elements on the diagonal), and then this (negative) element is used to compute DSTOP, the stopping value. But then we get a negative DSTOP,

      IF( TOL.LT.ZERO ) THEN
         DSTOP = N * DLAMCH( 'Epsilon' ) * AJJ
      ELSE
         DSTOP = TOL
      END IF

and then I stop even thinking on what would happen next. It seems that the code continues and does produce NAN in the first column. Why not. But I think we should catch this case and try to handle it in a different way. Maybe.

** Bug fix **

Replace
      IF( AJJ.EQ.ZERO.OR.DISNAN( AJJ ) ) THEN
With
      IF( AJJ.LE.ZERO.OR.DISNAN( AJJ ) ) THEN
So that if there is any nonpositive element or any NAN on the diagonal of A, we return right away with RANK=0 and INFO=1.

** Email from Nick Higham **

[ Julien wrote ] So that if there is any nonpositive element or any NAN on the diagonal of A, we return right away with RANK=0 and INFO=1?

[ Nick wrote ]

I think so.
This is justified by the documentation, which says

   INFO is INTEGER
   If INFO = -K, the K-th argument had an illegal value,
   = 0: algorithm completed successfully, and
   > 0: the matrix A is either rank deficient with computed rank
   as returned in RANK, or is indefinite.

We are in the indefinite case if any diagonal element is negative.
Actually, that is not quite true: we are in the "not positive definite"
case.  We can only say "indefinite" if there are both positive *and* negative
diagonal entries!  However, if that is going to be said then it would be
necessary to check that all other LAPACK routines are similarly precise in
their wording, so it's simplest, and not too misleading,
to stay with "indefinite".

9 years agoCorrecting Bug111 reported by Kevin Squire on LAPACK Mailing - August 15th 2013
julie [Wed, 18 Mar 2015 05:09:27 +0000 (05:09 +0000)]
Correcting Bug111 reported by Kevin Squire on LAPACK Mailing - August 15th 2013

From Kevin: We were following the recommendation for RWORK size calculation from
the ZGESDD header here ( http://netlib.org/lapack/complex16/zgesdd.f), and were
finding that when INFO=N, the size was insufficient, and generally led to a segmentation fault.

The current definition says that "If JOBZ = N, LRWORK >= 5*min(M,N)", but we found
that the minimum size needed to be at least 7*min(M,N). This was through experimentation
 and comparison with code from Numpy and Octave, so you may wish to verify this through the code.

JL: bug confirmed and reproduced - correction in comments
If JOBZ = N, LRWORK >= 7*min(M,N)

9 years agoCorrect issue reported by Elena (Oracle)
julie [Tue, 17 Mar 2015 14:53:16 +0000 (14:53 +0000)]
Correct issue reported by Elena (Oracle)
Add SIGMA and EPS in a list of arguments and describe them in the "Arguments" section

9 years agoCorrection reported by Elena (Oracle)
julie [Fri, 6 Mar 2015 04:31:28 +0000 (04:31 +0000)]
Correction reported by Elena (Oracle)

LAPACK 3.5.0 [cz]hgeqz.f compute eigenvalues differently depending on whether
eigenvectors are requested or not. Since the computation is different, eigenvalues
are not bit-by-bit identical whether eigenvectors are requested or not.

This fix enables to have [cz]hgeqz.f behaving more of the same whether eigenvectors
are requested or not.

Note: The fact that eigenvalues are bit-by-bit identical whether eigenvectors are
requested or not is checked in the LAPACK test suite.

9 years agoCorrect two ". GT." to " .GT.". Bug reported by email ny Willem Vermin "Minor
langou [Mon, 2 Mar 2015 16:04:35 +0000 (16:04 +0000)]
Correct two ". GT." to " .GT.". Bug reported by email ny Willem Vermin "Minor
issue in dorbdb.f and sorbdb.f" on Friday, February 27, 2015 at 4:03 AM.

9 years agoFix wrong update when the right Schur vector is not computed.
philippe.theveny [Wed, 25 Feb 2015 23:40:47 +0000 (23:40 +0000)]
Fix wrong update when the right Schur vector is not computed.

9 years agoAdd back cggglm to Makefile...build goes fine
julie [Wed, 25 Feb 2015 03:55:26 +0000 (03:55 +0000)]
Add back cggglm to Makefile...build goes fine