eth-cscs / DLA-Future

DLA-Future
https://eth-cscs.github.io/DLA-Future/master/
BSD 3-Clause "New" or "Revised" License
64 stars 14 forks source link

Fix build problem with clang due to nested structure definition #1027

Closed albestro closed 11 months ago

albestro commented 11 months ago

Fix #1017

@gulivarese can you please check if this works now?

albestro commented 11 months ago

cscs-ci run

codecov-commenter commented 11 months ago

Codecov Report

Merging #1027 (b97b44c) into master (85befe9) will increase coverage by 0.01%. Report is 2 commits behind head on master. The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master    #1027      +/-   ##
==========================================
+ Coverage   94.10%   94.11%   +0.01%     
==========================================
  Files         146      146              
  Lines        8843     8843              
  Branches     1121     1121              
==========================================
+ Hits         8322     8323       +1     
  Misses        326      326              
+ Partials      195      194       -1     
Files Coverage Δ
include/dlaf/eigensolver/tridiag_solver/merge.h 99.18% <100.00%> (ø)

... and 1 file with indirect coverage changes

msimberg commented 11 months ago

The error (without this fix) seems to reproduce here as well: https://gitlab.com/cscs-ci/ci-testing/webhook-ci/mirrors/4700071344751697/7514005670787789/-/jobs/5453597110#L416 (#1028).

rasolca commented 11 months ago

cscs-ci run