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 SYMBOLPREFIX and SYMBOLSUFFIX makefile options #459

Merged
merged 1 commit into from
Oct 25, 2014

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Oct 25, 2014

for adding a prefix or suffix to all exported symbol names in the shared library
Useful to avoid conflicts with other BLAS libraries, especially when using
64 bit integer interfaces in OpenBLAS

Note that since OSX does not have the objcopy utility, setting these options
to non-empty values on Mac requires the objconv tool, available (GPL license)
from http://www.agner.org/optimize/#objconv
(It is only run as a command-line application to change the symbol names at OpenBLAS-build-time so I don't think the GPL license is a problem, and if the options are not set then objconv is not needed.)

Refer to JuliaLang/julia#8734 and JuliaLang/julia#4923 for background motivation.

for adding a prefix or suffix to all exported symbol names in the shared library
Useful to avoid conflicts with other BLAS libraries, especially when using
64 bit integer interfaces in OpenBLAS

Note that since OSX does not have the objcopy utility, setting these options
to non-empty values on Mac requires the objconv tool, available (GPL license)
from http://www.agner.org/optimize/#objconv
xianyi added a commit that referenced this pull request Oct 25, 2014
add SYMBOLPREFIX and SYMBOLSUFFIX makefile options
@xianyi xianyi merged commit cbb23c4 into OpenMathLib:develop Oct 25, 2014
@tkelman tkelman deleted the symbol-rename branch October 25, 2014 11:59
@tkelman
Copy link
Contributor Author

tkelman commented Oct 25, 2014

Great, thanks @xianyi! Ping me if you end up needing to make any changes.

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.

2 participants