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

Enable "WIP: Wrap BLIS" with reference LAPACK #498

Open
wants to merge 1 commit into
base: blis
Choose a base branch
from

Conversation

jd-foster
Copy link

@jd-foster jd-foster commented May 7, 2024

Based on #431, this demonstrates how to get the BLIS extension working using libblastrampoline combined with LAPACK_jll (based on Reference LAPACK 3.11 https://github.com/Reference-LAPACK/lapack/releases/tag/v3.11)

Note that suffix_hint="64_", in BLAS.lbt_forward enables ILP versions.

@jd-foster jd-foster mentioned this pull request May 7, 2024
@jd-foster
Copy link
Author

The MWE as it stands needs to include LAPACK_jll now:

using LinearSolve, blis_jll, LAPACK_jll

A = rand(4, 4)
b = rand(4)
prob = LinearProblem(A, b)
sol = solve(prob,LinearSolve.BLISLUFactorization())
sol.u

@ChrisRackauckas
Copy link
Member

If you use LBT though then you're changing global settings.

@jd-foster
Copy link
Author

jd-foster commented May 8, 2024

I see now, you're trying to avoid LBT by going directly like you do with MKL?

to avoid allocations and does not require libblastrampoline.

@ChrisRackauckas
Copy link
Member

Yes, we'd prefer not to trigger global changes and instead be as local as possible.

@jd-foster
Copy link
Author

We'll see if we can get JuliaPackaging/Yggdrasil#7660 done then.

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.

2 participants