Skip to content
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

[lapacke] New port #42345

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

[lapacke] New port #42345

wants to merge 4 commits into from

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Nov 25, 2024

Alternative to #41911.

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 25, 2024

osx: BLAS and LAPACK by Accelerate framework. tmglib missing from lapacke.
windows: import lib named ...dll.a instead of ...lib. Dynamic library linkage also for static tripelts.

@Cheney-W Cheney-W added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Nov 25, 2024
@Neumann-A
Copy link
Contributor

Why do you need a Fortran Compiler?

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 25, 2024

Why do you need a Fortran Compiler?

In lapack-reference 0.11.0, the build system is tightly coupled to Fortran.
(That's one thing to re-visit before un-drafting.)

@@ -0,0 1,30 @@
diff --git a/CMakeLists.txt b/CMakeLists.txt
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks similar to what I do in #37409. I just print the C & Fortran linkage and then add it to the PC files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants