From 7f26615ec20e0986f977d5d367006af28b08a4be Mon Sep 17 00:00:00 2001 From: Antonio Sanchez Date: Tue, 22 Apr 2025 15:02:37 -0700 Subject: [PATCH] Fix commainitializer noexcept test. The `noexcept` identifier was accidentally removed in !1746, breaking the commainitializer tests, which do rely on Eigen throwing an exception. --- Eigen/src/Core/CommaInitializer.h | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Eigen/src/Core/CommaInitializer.h b/Eigen/src/Core/CommaInitializer.h index 4b9ed7735..c41411790 100644 --- a/Eigen/src/Core/CommaInitializer.h +++ b/Eigen/src/Core/CommaInitializer.h @@ -90,7 +90,13 @@ struct CommaInitializer { return *this; } - EIGEN_DEVICE_FUNC inline ~CommaInitializer() { finished(); } + EIGEN_DEVICE_FUNC inline ~CommaInitializer() +#if defined VERIFY_RAISES_ASSERT && (!defined EIGEN_NO_ASSERTION_CHECKING) && defined EIGEN_EXCEPTIONS + noexcept(false) // Eigen::eigen_assert_exception +#endif + { + finished(); + } /** \returns the built matrix once all its coefficients have been set. * Calling finished is 100% optional. Its purpose is to write expressions -- GitLab