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

@compat(BLAS.BlasInt(5)) #104

Closed
dlfivefifty opened this issue Jun 17, 2015 · 4 comments
Closed

@compat(BLAS.BlasInt(5)) #104

dlfivefifty opened this issue Jun 17, 2015 · 4 comments

Comments

@dlfivefifty
Copy link

This should be BLAS.blas_int(5) in 0.3.

@stevengj
Copy link
Member

Since this isn't even documented in Julia, I'm not sure it really belongs in Compat; you could always put the requisite BlasInt function in a module where you need this compatibility.

@dlfivefifty
Copy link
Author

I ended up removing the BlasInt conversion call: regular ints seem to work.

The usage was a wrapper for the LAPack routine zhseqr_, which mimiced the Base/LinAlg/LAPack.jl implementations

Cheers,
Sheehan

On 25 Jun 2015, at 3:26 pm, Steven G. Johnson notifications@github.com wrote:

Since this isn't even documented in Julia, I'm not sure it really belongs in Compat; you could always put the requisite BlasInt function in a module where you need this compatibility.


Reply to this email directly or view it on GitHub #104 (comment).

@ivarne ivarne closed this as completed Jun 25, 2015
@stevengj
Copy link
Member

ccall will do the conversions for you, but you need to declare the correct type (BlasInt) in the argument list. (Int may or may not work depending on what BLAS implementation you link.)

@dlfivefifty
Copy link
Author

Yes I left the declaration as BlasInt, which works in both 0.4 and 0.3

Sent from my iPad

On 26 Jun 2015, at 12:23 am, Steven G. Johnson notifications@github.com wrote:

ccall will do the conversions for you, but you need to declare the correct type (BlasInt) in the argument list. (Int may or may not work depending on what BLAS implementation you link.)


Reply to this email directly or view it on GitHub.

martinholters pushed a commit to martinholters/Compat.jl that referenced this issue Jul 13, 2016
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

No branches or pull requests

3 participants