-
Notifications
You must be signed in to change notification settings - Fork 482
Update Function Arguments in ?gees, ?geesx, ?gges, ?ggesx, and ?gges3
#1173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Update Function Arguments in ?gees, ?geesx, ?gges, ?ggesx, and ?gges3
#1173
Conversation
|
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: testing: Where I can find where LAPACK calls are located? |
|
Call tree looks valid. I have no idea why test failed. |
|
@langou, could you please add |
|
I added one for .github/workflows/cmake.yml (seems like other 'ctest' had it.) |
What is about continuous-integration/appveyor/pr? |
|
Ok. Found where it is :-) |
|
No, I definitely has no idea why appveyor fails. |
6351489 to
53c4803
Compare
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