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

Allow compilation with OpenMP 3.x #380

Merged
merged 2 commits into from
Apr 15, 2021

Conversation

martin-frbg
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Dec 31, 2019

Codecov Report

Merging #380 into master will increase coverage by 1.58%.
The diff coverage is 69.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #380      +/-   ##
==========================================
+ Coverage   81.66%   83.24%   +1.58%     
==========================================
  Files        1847     1808      -39     
  Lines      180341   170181   -10160     
==========================================
- Hits       147267   141664    -5603     
+ Misses      33074    28517    -4557
Impacted Files Coverage Δ
SRC/cgelqf.f 100% <ø> (ø) ⬆️
SRC/dgelq2.f 100% <ø> (ø) ⬆️
SRC/sgeqrf.f 95.65% <ø> (ø) ⬆️
SRC/dgeqr2p.f 100% <ø> (ø) ⬆️
SRC/sgeqr2.f 100% <ø> (ø) ⬆️
SRC/cgelq.f 86.88% <ø> (ø) ⬆️
SRC/dgeqr.f 86.88% <ø> (ø) ⬆️
SRC/cgelq2.f 100% <ø> (ø) ⬆️
SRC/zgeqr.f 86.88% <ø> (ø) ⬆️
SRC/cgeqr2.f 100% <ø> (ø) ⬆️
... and 1239 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12d8253...3e5c803. Read the comment docs.

@weslleyspereira
Copy link
Collaborator

Hi @martin-frbg. Does it still make sense to merge this PR? It seems good to me.

@martin-frbg
Copy link
Collaborator Author

I think so, although I never saw a reaction here - this has been in OpenBLAS for more than a year now

@langou langou merged commit 2614d23 into Reference-LAPACK:master Apr 15, 2021
christoph-conrads pushed a commit to christoph-conrads/lapack that referenced this pull request May 23, 2021
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