Skip to content

Clean up stableNorm

Charles Schlosserrequested to merge
chuckyschluz/eigen-chucky:eigen_stable_norm into master

Reference issue

What does this implement/fix?

Suppresses irritating maybe-uninitialized warning on gcc. ../Eigen/src/Core/CoreEvaluators.h:1004:48: warning: '*((void*)&<anonymous> +32)' may be used uninitialized in this function [-Wmaybe-uninitialized] Breaking the computation into blocks doesn't appear to add value, and appears to make an extra copy of the entire expression. The bulk of the work is this line Scalar maxCoeff = bl.cwiseAbs().maxCoeff(); which is already lazily evaluated, vectorized, etc. The warning is probably caused by the object in const Ref which is not initialized.

Additional information

Merge request reports