Skip to content

Commit

Permalink
Update BLAS.vendor() to return :lbt
Browse files Browse the repository at this point in the history
`vendor()` returns `:lbt`
`libblas_name` and `liblapack_name` are set to "libblastrampoline"
  • Loading branch information
ViralBShah committed Mar 3, 2022
1 parent 8800a80 commit bf6d9de
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
4 changes: 4 additions & 0 deletions base/Base.jl
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,10 @@ end
include(strcat((length(Core.ARGS)>=2 ? Core.ARGS[2] : ""), "build_h.jl")) # include($BUILDROOT/base/build_h.jl)
include(strcat((length(Core.ARGS)>=2 ? Core.ARGS[2] : ""), "version_git.jl")) # include($BUILDROOT/base/version_git.jl)

# These used to be in build_h.jl and are retained for backwards compatibility
const libblas_name = "libblastrampoline"
const liblapack_name = "libblastrampoline"

# numeric operations
include("hashing.jl")
include("rounding.jl")
Expand Down
13 changes: 2 additions & 11 deletions stdlib/LinearAlgebra/src/blas.jl
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ using ..LinearAlgebra: libblastrampoline, BlasReal, BlasComplex, BlasFloat, Blas

include("lbt.jl")

vendor() = :lbt

"""
get_config()
Expand All @@ -77,17 +79,6 @@ Return an object representing the current `libblastrampoline` configuration.
"""
get_config() = lbt_get_config()

# We hard-lock `vendor()` to `openblas(64)` here to satisfy older code, but all new code should use
# `get_config()` since it is now possible to have multiple vendors loaded at once.
function vendor()
Base.depwarn("`vendor()` is deprecated, use `BLAS.get_config()` and inspect the output instead", :vendor; force=true)
if USE_BLAS64
return :openblas64
else
return :openblas
end
end

if USE_BLAS64
macro blasfunc(x)
return Expr(:quote, Symbol(x, "64_"))
Expand Down

0 comments on commit bf6d9de

Please sign in to comment.