GitHub-Proxy/eigen
1
0
Fork 0
mirror of https://gitlab.com/libeigen/eigen.git synced 2025-09-13 01:43:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "BDCSVD fix -Wmaybe-uninitialized"

Browse Source 
This reverts merge request !1649
This commit is contained in:
Rasmus Munk Larsen 2024-08-05 18:17:01 +00:00
parent d14b0a4e53
commit 1dcae7cefc
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Eigen/src/SVD/BDCSVD.h
View File

@ -52,8 +52,9 @@ struct traits<BDCSVD<MatrixType_, Options> > : svd_traits<MatrixType_, Options>
template <typename MatrixType, int Options>
struct allocate_small_svd {
static void run(JacobiSVD<MatrixType, Options>& smallSvd, Index rows, Index cols, unsigned int) {
internal::construct_at(&smallSvd, rows, cols);
static void run(JacobiSVD<MatrixType, Options>& smallSvd, Index rows, Index cols, unsigned int computationOptions) {
(void)computationOptions;
smallSvd = JacobiSVD<MatrixType, Options>(rows, cols);
}
};
@ -63,7 +64,7 @@ EIGEN_DISABLE_DEPRECATED_WARNING
template <typename MatrixType>
struct allocate_small_svd<MatrixType, 0> {
static void run(JacobiSVD<MatrixType>& smallSvd, Index rows, Index cols, unsigned int computationOptions) {
internal::construct_at(&smallSvd, rows, cols, computationOptions);
smallSvd = JacobiSVD<MatrixType>(rows, cols, computationOptions);
}
};

3
Eigen/src/SVD/JacobiSVD.h
View File

@ -553,8 +553,7 @@ class JacobiSVD : public SVDBase<JacobiSVD<MatrixType_, Options_> > {
* \deprecated Will be removed in the next major Eigen version. Options should
* be specified in the \a Options template parameter.
*/
// EIGEN_DEPRECATED // this constructor is used to allocate memory in BDCSVD
JacobiSVD(Index rows, Index cols, unsigned int computationOptions) {
EIGEN_DEPRECATED JacobiSVD(Index rows, Index cols, unsigned int computationOptions) {
internal::check_svd_options_assertions<MatrixType, Options>(computationOptions, rows, cols);
allocate(rows, cols, computationOptions);
}