Skip to content

SRC: use LSAME for TRANS checks in gttrs#1239

Open
nakatamaho wants to merge 4 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/gttrs-lsame-trans
Open

SRC: use LSAME for TRANS checks in gttrs#1239
nakatamaho wants to merge 4 commits intoReference-LAPACK:masterfrom
nakatamaho:cleanup/gttrs-lsame-trans

Conversation

@nakatamaho
Copy link
Copy Markdown
Contributor

This PR replaces direct case-sensitive TRANS character comparisons in the GTTRS routines with LSAME.

Files updated:

SRC/sgttrs.f
SRC/dgttrs.f
SRC/cgttrs.f
SRC/zgttrs.f

Summary:

  • Replace direct N/n, T/t, and C/c TRANS checks with LSAME.
  • Use LSAME( TRANS, 'T' ) for the complex ITRANS branch.
  • Add LSAME as a logical external function.

Validation:

  • Compiled sgttrs.f, dgttrs.f, cgttrs.f, and zgttrs.f with gfortran -O2 -c.
  • Ran git diff --check for the staged changes before commit.

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