Skip to content

Update Function Arguments in ?gees, ?geesx, ?gges, ?ggesx, and ?gges3#1173

Merged
langou merged 4 commits intoReference-LAPACK:masterfrom
foxtran:update/function-arguments
Apr 19, 2026
Merged

Update Function Arguments in ?gees, ?geesx, ?gges, ?ggesx, and ?gges3#1173
langou merged 4 commits intoReference-LAPACK:masterfrom
foxtran:update/function-arguments

Conversation

@foxtran
Copy link
Copy Markdown
Contributor

@foxtran foxtran commented Nov 30, 2025

Instead of using external keyword for Function Arguments, this PR introduces explicit interfaces for Function Arguments. The code should be more safe now, and source code reflect documented behaviour. Later, it will help a lot to generate proper interfaces for these functions.

BIND(C) is used since code can be called from other than Fortran languages.

Checklist

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Nov 30, 2025

Hmmm.. I used flang 21.1.6 on MacOS Ventura 13.7.8 to check what wrong in CI, and all tests were passed.

Configure command:

cmake -G "Ninja" -DCMAKE_Fortran_COMPILER=flang -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=ON -DCMAKE_Fortran_FLAGS="-g" ..

testing:

ctest . --parallel 4

Where I can find where LAPACK calls are located?

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Nov 30, 2025

Call tree looks valid. I have no idea why test failed.

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Dec 1, 2025

@langou, could you please add --output-on-failure flag to ctest command into CI?

@langou
Copy link
Copy Markdown
Contributor

langou commented Dec 1, 2025

I added one for .github/workflows/cmake.yml (seems like other 'ctest' had it.)
Let me know if you want it at other places.
Note: I'll need a "maintainer" to approve my PR so as to be able to merge it.

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Dec 1, 2025

Let me know if you want it at other places.

What is about continuous-integration/appveyor/pr?

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Dec 1, 2025

Ok. Found where it is :-)

@foxtran
Copy link
Copy Markdown
Contributor Author

foxtran commented Dec 1, 2025

No, I definitely has no idea why appveyor fails.

@foxtran foxtran force-pushed the update/function-arguments branch 2 times, most recently from 53c4803 to efe84a1 Compare December 3, 2025 12:45
@martin-frbg
Copy link
Copy Markdown
Collaborator

Could be it's getting miscompiled, because the CI job running on Appveyor appears to be still using flang 11 from four years ago when the new flang was barely able to compile again.
(I had to clamp the version number around then to keep the CI running at all, because conda was acting up and tried to update both flang and itself with the dependencies all wrong)

@martin-frbg
Copy link
Copy Markdown
Collaborator

closing&reopening to trigger a new CI run

@martin-frbg martin-frbg reopened this Apr 18, 2026
@langou langou merged commit 09e8a7b into Reference-LAPACK:master Apr 19, 2026
12 checks passed
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.

Provide explicit interfaces for external functions passed as arguments

3 participants