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

Add OPENBLAS_NO_AVX build option to disable AVX kernels #7476

Merged
merged 1 commit into from
Jul 2, 2014

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Jun 30, 2014

useful for compiling with OPENBLAS_DYNAMIC_ARCH = 1 using old binutils versions

ref https://groups.google.com/forum/#!topic/julia-users/t9oR1Z_ziM0 and #7240 and #7363 (comment)

useful for compiling with OPENBLAS_DYNAMIC_ARCH = 1 using old binutils versions
ViralBShah added a commit that referenced this pull request Jul 2, 2014
Add OPENBLAS_NO_AVX build option to disable AVX kernels
@ViralBShah ViralBShah merged commit e63ebd9 into JuliaLang:master Jul 2, 2014
@tkelman tkelman deleted the openblas-noavx branch July 2, 2014 04:54
@johnmyleswhite
Copy link
Member

Should we expose NO_AVX2 as well? On a CentOS 6 system, I found that the flag that mattered most was NO_AVX2, which seemed not to be implied by NO_AVX.

@tkelman
Copy link
Contributor Author

tkelman commented Jul 25, 2014

Damn, maybe I should've looked closer at the OpenBLAS commit that added NO_AVX2, I was under the assumption NO_AVX would imply NO_AVX2. I'll go add that to deps/Makefile then, should also help with #7653

tkelman added a commit that referenced this pull request Jul 25, 2014
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.

3 participants