Skip to content

Fix macro used in LAPACKE_zgesvdq#458

Merged
langou merged 1 commit intoReference-LAPACK:masterfrom
nforro:master
Oct 22, 2020
Merged

Fix macro used in LAPACKE_zgesvdq#458
langou merged 1 commit intoReference-LAPACK:masterfrom
nforro:master

Conversation

@nforro
Copy link
Copy Markdown
Contributor

@nforro nforro commented Oct 22, 2020

This looks like a copy-paste error. gcc emits a warning about it:

lapacke_zgesvdq.c: In function 'LAPACKE_zgesvdq':
../include/lapacke.h:50:42: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

This looks like a copy-paste error. gcc emits a warning about it:

lapacke_zgesvdq.c: In function 'LAPACKE_zgesvdq':
../include/lapacke.h:50:42: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

Signed-off-by: Nikola Forró <nforro@redhat.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 22, 2020

Codecov Report

Merging #458 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #458   +/-   ##
=======================================
  Coverage   81.86%   81.86%           
=======================================
  Files        1863     1863           
  Lines      181096   181096           
=======================================
  Hits       148263   148263           
  Misses      32833    32833           

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 c3a0b79...3c7d0f0. Read the comment docs.

@langou
Copy link
Copy Markdown
Contributor

langou commented Oct 22, 2020

Thanks!

@langou langou merged commit 4fbb4f8 into Reference-LAPACK:master Oct 22, 2020
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.

2 participants