devtools/eigen
1
0
Fork 0
mirror of https://gitlab.com/libeigen/eigen.git synced 2026-04-10 11:34:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert accidental changes from !2212 squash merge

Browse Source 
libeigen/eigen!2214

Co-authored-by: Rasmus Munk Larsen <rmlarsen@gmail.com>
This commit is contained in:
Rasmus Munk Larsen
2026-02-25 08:31:41 -08:00
parent 38f0f42755
commit ea25ea52bb
13 changed files with 331 additions and 1352 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
test/CMakeLists.txt
View File

@@ -179,7 +179,6 @@ ei_add_test(numext)
ei_add_test(sizeof)
ei_add_test(dynalloc)
ei_add_test(nomalloc)
ei_add_test(noresize)
ei_add_test(first_aligned)
ei_add_test(type_alias)
ei_add_test(nullary)
@@ -187,17 +186,6 @@ ei_add_test(mixingtypes)
ei_add_test(float_conversion)
ei_add_test(io)
ei_add_test(packetmath "-DEIGEN_FAST_MATH=1")
# Generic clang vector backend tests for different vector sizes.
include(CheckCXXSourceCompiles)
check_cxx_source_compiles("
typedef float v4sf __attribute__((ext_vector_type(4)));
int main() { return __builtin_vectorelements(v4sf{}); }
" COMPILER_SUPPORTS_VECTOR_EXTENSIONS)
if(COMPILER_SUPPORTS_VECTOR_EXTENSIONS)
ei_add_test(packetmath_generic_16 "-DEIGEN_FAST_MATH=1")
ei_add_test(packetmath_generic_32 "-DEIGEN_FAST_MATH=1")
ei_add_test(packetmath_generic_64 "-DEIGEN_FAST_MATH=1")
endif()
ei_add_test(packet_segment)
ei_add_test(vectorization_logic)
ei_add_test(basicstuff)

2
test/evaluators.cpp
View File

@@ -33,8 +33,6 @@ EIGEN_STRONG_INLINE DstXprType& copy_using_evaluator(const PlainObjectBase<DstXp
eigen_assert((dst.size() == 0 || (IsVectorAtCompileTime ? (dst.size() == src.size())
: (dst.rows() == src.rows() && dst.cols() == src.cols()))) &&
"Size mismatch. Automatic resizing is disabled because EIGEN_NO_AUTOMATIC_RESIZING is defined");
// Allow resizing of default-constructed (empty) destinations.
if (dst.size() == 0) dst.const_cast_derived().resizeLike(src.derived());
#else
dst.const_cast_derived().resizeLike(src.derived());
#endif

110
test/noresize.cpp
View File

@@ -1,110 +0,0 @@
// This file is part of Eigen, a lightweight C++ template library
// for linear algebra.
//
// Copyright (C) 2026 Rasmus Munk Larsen <rmlarsen@gmail.com>
//
// This Source Code Form is subject to the terms of the Mozilla
// Public License v. 2.0. If a copy of the MPL was not distributed
// with this file, You can obtain one at http://mozilla.org/MPL/2.0/.
// Must be defined before including any Eigen headers.
#define EIGEN_NO_AUTOMATIC_RESIZING
#include "main.h"
// Helper to create a random matrix respecting compile-time fixed dimensions.
template <typename MatrixType>
MatrixType random_matrix() {
enum { RowsAtCompileTime = MatrixType::RowsAtCompileTime, ColsAtCompileTime = MatrixType::ColsAtCompileTime };
Index rows = (RowsAtCompileTime == Dynamic) ? internal::random<Index>(1, 10) : Index(RowsAtCompileTime);
Index cols = (ColsAtCompileTime == Dynamic) ? internal::random<Index>(1, 10) : Index(ColsAtCompileTime);
return MatrixType::Random(rows, cols);
}
template <typename MatrixType>
void noresize_assign_to_empty() {
MatrixType src = random_matrix<MatrixType>();
// Assigning to a default-constructed (empty) destination should work.
MatrixType dst;
dst = src;
VERIFY_IS_EQUAL(dst.rows(), src.rows());
VERIFY_IS_EQUAL(dst.cols(), src.cols());
VERIFY_IS_APPROX(dst, src);
}
template <typename MatrixType>
void noresize_assign_expression_to_empty() {
MatrixType a = random_matrix<MatrixType>();
MatrixType b(a.rows(), a.cols());
b.setRandom();
// Assigning an expression to an empty destination should work.
MatrixType dst;
dst = a + b;
VERIFY_IS_EQUAL(dst.rows(), a.rows());
VERIFY_IS_EQUAL(dst.cols(), a.cols());
VERIFY_IS_APPROX(dst, a + b);
}
template <typename MatrixType>
void noresize_construct_from_expression() {
MatrixType a = random_matrix<MatrixType>();
// Construction from an expression should work.
MatrixType dst = a * 2;
VERIFY_IS_EQUAL(dst.rows(), a.rows());
VERIFY_IS_EQUAL(dst.cols(), a.cols());
VERIFY_IS_APPROX(dst, a * 2);
}
template <typename MatrixType>
void noresize_col_access() {
MatrixType src = random_matrix<MatrixType>();
// Assigning to empty, then accessing columns should work.
MatrixType dst;
dst = src;
for (Index j = 0; j < src.cols(); ++j) {
VERIFY_IS_APPROX(dst.col(j), src.col(j));
}
}
template <typename MatrixType>
void noresize_size_mismatch() {
enum { RowsAtCompileTime = MatrixType::RowsAtCompileTime, ColsAtCompileTime = MatrixType::ColsAtCompileTime };
Index rows = (RowsAtCompileTime == Dynamic) ? internal::random<Index>(2, 10) : Index(RowsAtCompileTime);
Index cols = (ColsAtCompileTime == Dynamic) ? internal::random<Index>(2, 10) : Index(ColsAtCompileTime);
MatrixType src = MatrixType::Random(rows, cols);
// Create a destination with at least one mismatched dynamic dimension.
Index dst_rows = (RowsAtCompileTime == Dynamic) ? rows + 1 : rows;
Index dst_cols = (ColsAtCompileTime == Dynamic) ? cols + 1 : cols;
MatrixType dst = MatrixType::Random(dst_rows, dst_cols);
// Assigning to a non-empty destination with different size should assert.
VERIFY_RAISES_ASSERT(dst = src);
}
EIGEN_DECLARE_TEST(noresize) {
CALL_SUBTEST_1(noresize_assign_to_empty<MatrixXf>());
CALL_SUBTEST_1(noresize_assign_to_empty<MatrixXd>());
CALL_SUBTEST_1(noresize_assign_to_empty<MatrixXcf>());
CALL_SUBTEST_1(noresize_assign_to_empty<MatrixXcd>());
CALL_SUBTEST_2(noresize_assign_to_empty<ArrayXXd>());
CALL_SUBTEST_2(noresize_assign_to_empty<ArrayXXcd>());
CALL_SUBTEST_3(noresize_assign_to_empty<VectorXf>());
CALL_SUBTEST_3(noresize_assign_to_empty<RowVectorXd>());
CALL_SUBTEST_4(noresize_assign_expression_to_empty<MatrixXd>());
CALL_SUBTEST_4(noresize_assign_expression_to_empty<ArrayXXd>());
CALL_SUBTEST_5(noresize_construct_from_expression<MatrixXd>());
CALL_SUBTEST_5(noresize_construct_from_expression<ArrayXXd>());
CALL_SUBTEST_6(noresize_col_access<MatrixXd>());
CALL_SUBTEST_6(noresize_col_access<MatrixXf>());
CALL_SUBTEST_7(noresize_size_mismatch<MatrixXd>());
CALL_SUBTEST_7(noresize_size_mismatch<MatrixXf>());
CALL_SUBTEST_7(noresize_size_mismatch<VectorXd>());
}

4
test/packetmath_generic_16.cpp
View File

@@ -1,4 +0,0 @@
// Force the generic clang vector backend with 16-byte vectors.
#define EIGEN_VECTORIZE_GENERIC 1
#define EIGEN_GENERIC_VECTOR_SIZE_BYTES 16
#include "packetmath.cpp"

4
test/packetmath_generic_32.cpp
View File

@@ -1,4 +0,0 @@
// Force the generic clang vector backend with 32-byte vectors.
#define EIGEN_VECTORIZE_GENERIC 1
#define EIGEN_GENERIC_VECTOR_SIZE_BYTES 32
#include "packetmath.cpp"

4
test/packetmath_generic_64.cpp
View File

@@ -1,4 +0,0 @@
// Force the generic clang vector backend with 64-byte vectors.
#define EIGEN_VECTORIZE_GENERIC 1
#define EIGEN_GENERIC_VECTOR_SIZE_BYTES 64
#include "packetmath.cpp"