Skip to content

TESTING/LIN: use LSAME for UPLO checks in latsp and latsy#1241

Open
nakatamaho wants to merge 6 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/testing-lin-latsp-latsy-lsame-uplo
Open

TESTING/LIN: use LSAME for UPLO checks in latsp and latsy#1241
nakatamaho wants to merge 6 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/testing-lin-latsp-latsy-lsame-uplo

Conversation

@nakatamaho
Copy link
Copy Markdown
Contributor

This PR replaces direct UPLO character comparisons in the LATSP and LATSY test helpers with LSAME.

Files updated:

TESTING/LIN/clatsp.f
TESTING/LIN/zlatsp.f
TESTING/LIN/clatsy.f
TESTING/LIN/zlatsy.f

Summary:

  • Replace UPLO.EQ.'U' with LSAME( UPLO, 'U' ).
  • Add LSAME as a logical external function.

…it tests

Reorder ZERO and ONE declarations and PARAMETER definitions
for consistency in the TESTING/EIG error-exit test sources.

No functional change intended.
Replace hard-coded floating-point constants with named constants in selected TESTING/EIG error-exit tests. This keeps the test setup code consistent with surrounding LAPACK test style.
Replace direct case-sensitive UPLO character comparisons in CPTTRS and ZPTTRS with LSAME. This matches the surrounding LAPACK convention for case-insensitive option checks.
Replace direct case-sensitive TRANS character comparisons in the GTTRS routines with LSAME. This keeps option handling consistent with the surrounding LAPACK convention for case-insensitive checks.
Replace direct UPLO character comparisons in the LALSD routines with LSAME. This keeps option handling consistent with the surrounding LAPACK convention for case-insensitive checks.
Replace direct UPLO character comparisons in the LATSP and LATSY test helpers with LSAME. This keeps option handling consistent with the LAPACK convention for case-insensitive checks.
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.

1 participant