diff options
| author |  Gael Guennebaud <g.gael@free.fr> | 2015-10-28 11:42:14 +0100 |
|---|---|---|
| committer | Gael Guennebaud <g.gael@free.fr> | 2015-10-28 11:42:14 +0100 |
| commit | 77ff3386b7d28c68c9e277e60f285ae1b3124b47 (patch) | |
| tree | a9e5cefe36dd832d1d1d7e0dbdcedf1c8c1b5b56 /test/redux.cpp | |
| parent | 827d8a9bad6f6c8a8e0211358b51c60db18a2cfb (diff) | |
Refactoring of the cost model:
- Dynamic is now an invalid value
- introduce a HugeCost constant to be used for runtime-cost values or arbitrarily huge cost
- add sanity checks for cost values: must be >=0 and not too large
This change provides several benefits:
- it fixes shortcoming is some cost computation where the Dynamic case was not properly handled.
- it simplifies cost computation logic, and should avoid future similar shortcomings.
- it allows to distinguish between different level of dynamic/huge/infinite cost
- it should enable further simplifications in the computation of costs (save compilation time)
Diffstat (limited to 'test/redux.cpp')
| -rw-r--r-- | test/redux.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test/redux.cpp b/test/redux.cpp index bfd9a8d50..6ddc59c18 100644 --- a/test/redux.cpp +++ b/test/redux.cpp @@ -74,7 +74,6 @@ template<typename MatrixType> void matrixRedux(const MatrixType& m) Matrix<Scalar, MatrixType::RowsAtCompileTime, MatrixType::RowsAtCompileTime> m2(rows,rows); m2.setRandom(); VERIFY_EVALUATION_COUNT( ((m1.matrix()*m1.matrix().transpose())+m2).sum(), (MatrixType::SizeAtCompileTime==Dynamic ? 1 : 0) ); - } template<typename VectorType> void vectorRedux(const VectorType& w) |