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

Bump OpenBLAS yet again #35041

Merged
merged 1 commit into from
Mar 8, 2020
Merged

Bump OpenBLAS yet again #35041

merged 1 commit into from
Mar 8, 2020

Conversation

Keno
Copy link
Member

@Keno Keno commented Mar 8, 2020

The previous patch I had was incomplete.

@ViralBShah
Copy link
Member

The number of files these things touch is a bit insane!

@Keno
Copy link
Member Author

Keno commented Mar 8, 2020

yeah, each checksum is a separate file because that's easy to check in a shell script, but we should probably do something more intelligent here, like have a tab-separated file like

OpenBlas+8 sha512 abcd...
OpenBlas+8 sha3 abcd...

Shouldn't be too bad to handle with standard unix tools.

@ViralBShah
Copy link
Member

wouldn't standard unix tools break on windows?

@Keno
Copy link
Member Author

Keno commented Mar 8, 2020

No, our build system already depends on them, so we assume their presence on windows (either through cygwin or msys).

@Keno Keno merged commit 17ad922 into master Mar 8, 2020
@Keno Keno deleted the kf/openblasbump branch March 8, 2020 01:43
@PallHaraldsson
Copy link
Contributor

PallHaraldsson commented Mar 9, 2020

Is there a reason to not go to the latest version (now 0.3.9)? Possibly this change was in the pipeline for a long time. I however see e.g. in latest version:

  • Fixed cpu autodetection of Goldmont+, Cannon Lake and Ice Lake

and

  • Improved speed of the AVX512 GEMM3M kernel on SkylakeX

ravibitsgoa pushed a commit to ravibitsgoa/julia that referenced this pull request Apr 9, 2020
KristofferC pushed a commit that referenced this pull request Apr 11, 2020
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.

None yet

3 participants