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

Change complex dot to use cblas to avoid problem with complex return type #5291

Merged
merged 1 commit into from
Jan 3, 2014

Conversation

andreasnoack
Copy link
Member

This should solve the problems discovered in #5283

@staticfloat
Copy link
Member

This fixes the issue on my computer, I'm going to merge. Thanks @andreasnoackjensen!

staticfloat added a commit that referenced this pull request Jan 3, 2014
Change complex dot to use cblas to avoid problem with complex return type
@staticfloat staticfloat merged commit 756a34a into master Jan 3, 2014
@andreasnoack andreasnoack deleted the anj/dotu branch January 7, 2014 13:03
@staticfloat
Copy link
Member

This is causing problems on CentOS when using system-provided blas. I appears that there are no cblas_ symbols in the system-provided blas. Do we need to worry about this, or is the answer to just not use system-provided blas libraries on CentOS? (I can probably find an OpenBLAS package somewhere)

@tkelman
Copy link
Contributor

tkelman commented Aug 19, 2014

You can try BLIS https://github.com/flame/blis/, they just added a cblas compatibility layer recently and it builds faster than OpenBLAS.

@staticfloat
Copy link
Member

I think if I'm going to compile something, I'll just compile OpenBLAS. I'm mostly checking to see if breaking compatibility with fortran-only BLAS libraries is something that we want. In fact, I should just open a new issue about this. #8062

@ViralBShah
Copy link
Member

Cblas is usually available separately.

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.

4 participants