Skip to content

COMP: Suppress GCC -Wmaybe-uninitialized in Eigen3 SelfadjointMatrixVector#5953

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
hjmjohnson:suppress-eigen-wmaybe-uninitialized
Mar 16, 2026
Merged

COMP: Suppress GCC -Wmaybe-uninitialized in Eigen3 SelfadjointMatrixVector#5953
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
hjmjohnson:suppress-eigen-wmaybe-uninitialized

Conversation

@hjmjohnson
Copy link
Member

Summary

  • Suppress false-positive GCC -Wmaybe-uninitialized warnings in Eigen3's SelfadjointMatrixVector.h on ARM (aarch64)
  • Uses #pragma GCC diagnostic push/pop scoped to the file, guarded by __GNUC__ and !__clang__

Test plan

  • ARM CI build passes without -Wmaybe-uninitialized warnings from this file
  • Non-ARM and Clang builds are unaffected (guards exclude them)

🤖 Generated with Claude Code

…ector

GCC on aarch64 (ARM) emits a false-positive -Wmaybe-uninitialized
warning at line 224 of SelfadjointMatrixVector.h. The dashboard script
(itk_common.cmake) treats any build warning as CI failure.

Add targeted GCC diagnostic pragmas to suppress the warning in this
file only, scoped to GCC (not Clang) via __GNUC__ / __clang__ guards.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added type:Compiler Compiler support or related warnings area:ThirdParty Issues affecting the ThirdParty module labels Mar 16, 2026
@hjmjohnson
Copy link
Member Author

Several attempts at suppressing build warnings were unsuccessful in

This approach seems to work.

@thewtex
Copy link
Member

thewtex commented Mar 16, 2026

@hjmjohnson thanks for keeping the CI green 💚

@hjmjohnson hjmjohnson merged commit 1e27859 into InsightSoftwareConsortium:main Mar 16, 2026
15 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ThirdParty Issues affecting the ThirdParty module type:Compiler Compiler support or related warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants