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

make AVX warnings optional #338

Closed
anyeli opened this issue Jan 22, 2014 · 2 comments
Closed

make AVX warnings optional #338

anyeli opened this issue Jan 22, 2014 · 2 comments

Comments

@anyeli
Copy link

anyeli commented Jan 22, 2014

The library, if compiled with DYNAMIC_ARCH=1, may print out something like this:

OpenBLAS : Your OS does not support AVX instructions. OpenBLAS is using Barcelona kernels as a fallback, which may give poorer performance.

The library user may not want to see this as,

  • it is not really an error; it does not terminate the program and does not cause incorrect results
  • there is nothing to do if upgrading is not an option
  • it may disrupt the user if used in an interactive program
  • it may break scripts that assume the program indicates errors by printing to standard error (e.g. some old Fortran programs)
  • it is just annoying if the user does not care :)

Could you please make this printout optional, as a compile time option, or even as a run time option? There are 6 places in driver/others/dynamic.c where a variant of this printout occurs.

xianyi added a commit that referenced this issue Jan 23, 2014
By default, OpenBLAS doesn't output the warning message. You can set
OPENBLAS_VERBOSE (e.g. export OPENBLAS_VERBOSE=1) to enable the warning
message on runtime.
@xianyi
Copy link
Collaborator

xianyi commented Jan 23, 2014

@anyeli , please try the develop branch.

@anyeli
Copy link
Author

anyeli commented Jan 28, 2014

Fix looks good, thanks!

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

2 participants