Skip to content

SRC: use LSAME for UPLO checks in lalsd#1240

Open
nakatamaho wants to merge 5 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/lalsd-lsame-uplo
Open

SRC: use LSAME for UPLO checks in lalsd#1240
nakatamaho wants to merge 5 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/lalsd-lsame-uplo

Conversation

@nakatamaho
Copy link
Copy Markdown
Contributor

This PR replaces direct UPLO character comparisons in the LALSD routines with LSAME.

Files updated:

SRC/slalsd.f
SRC/dlalsd.f
SRC/clalsd.f
SRC/zlalsd.f

Summary:

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

Validation:

  • Ran git diff --check for the staged changes before commit.
  • Confirmed full build and full TESTING pass.

…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.
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