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

[openblas] Add feature simplethread and thread #18265

Merged
merged 6 commits into from
Jun 10, 2021

Conversation

JackBoosY
Copy link
Contributor

@JackBoosY JackBoosY commented Jun 4, 2021

Add feature threads and simplethread that user can enable or disable them when building openblas.

Fixes #15698.

Already tested all features successfully on x86-windows and x64-windows-static.

@JackBoosY JackBoosY added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:internal This PR or Issue was filed by the vcpkg team. labels Jun 4, 2021
@JackBoosY
Copy link
Contributor Author

@TonyXiang8787 Can you please test this PR?
Install openblas[core] should be what you want.

@TonyXiang8787
Copy link

Tested!

root@19bb598141b2:~/vcpkg# ./vcpkg install openblas --dry-run
Computing installation plan...
The following packages will be built and installed:
    openblas[core,thread]:x64-linux -> 0.3.10#1
root@19bb598141b2:~/vcpkg# ./vcpkg install openblas[core] --dry-run
Computing installation plan...
The following packages will be built and installed:
    openblas[core]:x64-linux -> 0.3.10#1
root@19bb598141b2:~/vcpkg# ./vcpkg install openblas[thread] --dry-run
Computing installation plan...
The following packages will be built and installed:
    openblas[core,thread]:x64-linux -> 0.3.10#1
root@19bb598141b2:~/vcpkg# ./vcpkg install openblas[simplethread] --dry-run
Computing installation plan...
The following packages will be built and installed:
    openblas[core,simplethread,thread]:x64-linux -> 0.3.10#1

@JackBoosY
Copy link
Contributor Author

@TonyXiang8787 Please note only openblas built with core uses single thread.

@TonyXiang8787
Copy link

@JackBoosY I understand. The default behavior is kept the same (multi-threading) as current version.

@JackBoosY JackBoosY requested review from NancyLi1013 and PhoebeHui and removed request for NancyLi1013 June 7, 2021 02:13
ports/openblas/portfile.cmake Outdated Show resolved Hide resolved
ports/openblas/vcpkg.json Outdated Show resolved Hide resolved
ports/openblas/vcpkg.json Outdated Show resolved Hide resolved
versions/o-/openblas.json Outdated Show resolved Hide resolved
@PhoebeHui PhoebeHui added the info:reviewed Pull Request changes follow basic guidelines label Jun 7, 2021
@strega-nil-ms
Copy link
Contributor

Cool, thanks Jack!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[openblas] Support feature single-thread
4 participants