Skip to content

cblas: mark cblas_xerbla and F77_xerbla symbols as weak#521

Merged
langou merged 2 commits intoReference-LAPACK:masterfrom
veprbl:pr/cblas_use_weak_symbols
Mar 18, 2021
Merged

cblas: mark cblas_xerbla and F77_xerbla symbols as weak#521
langou merged 2 commits intoReference-LAPACK:masterfrom
veprbl:pr/cblas_use_weak_symbols

Conversation

@veprbl
Copy link
Copy Markdown
Contributor

@veprbl veprbl commented Mar 18, 2021

This should fix an issue reported by @bobrik in #440 (comment), although I discovered it was also appearing on x86_64-darwin. Thanks to @langou for the useful summary #440 (comment) .

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2021

Codecov Report

Merging #521 (b4123cb) into master (1827da0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #521   +/-   ##
=======================================
  Coverage   83.33%   83.33%           
=======================================
  Files        1820     1820           
  Lines      170893   170893           
=======================================
  Hits       142411   142411           
  Misses      28482    28482           

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 1827da0...b4123cb. Read the comment docs.

@weslleyspereira
Copy link
Copy Markdown
Collaborator

Hi! Thanks for working on that! Please see my comment here: #440 (comment)

@langou langou merged commit 8ebb2ad into Reference-LAPACK:master Mar 18, 2021
@veprbl veprbl deleted the pr/cblas_use_weak_symbols branch March 18, 2021 23:25
christoph-conrads pushed a commit to christoph-conrads/lapack that referenced this pull request May 23, 2021
…_symbols

cblas: mark cblas_xerbla and F77_xerbla symbols as weak
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