GitHub-Proxy/eigen
1
0
Fork 0
mirror of https://gitlab.com/libeigen/eigen.git synced 2025-06-04 18:54:00 +08:00
Code Issues Packages Projects Releases Wiki Activity

Optimize Ref<SparseMatrix> by removing useless default initialisation of SparseMapBase and SparseMatrix

Browse Source
This commit is contained in:
Gael Guennebaud 2015-10-06 11:57:03 +02:00
parent 9a070638de
commit 945b80c83e
2 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Eigen/src/SparseCore/SparseMap.h
View File

@ -120,6 +120,9 @@ class SparseMapBase<Derived,ReadOnlyAccessors>
/** Empty destructor */
inline ~SparseMapBase() {}
protected:
inline SparseMapBase() {}
};
template<typename Derived>
@ -172,6 +175,9 @@ class SparseMapBase<Derived,WriteAccessors>
/** Empty destructor */
inline ~SparseMapBase() {}
protected:
inline SparseMapBase() {}
};
template<typename MatScalar, int MatOptions, typename MatIndex, int Options, typename StrideType>

12
Eigen/src/SparseCore/SparseRef.h
View File

@ -182,8 +182,9 @@ class Ref<const SparseMatrix<MatScalar,MatOptions,MatIndex>, Options, StrideType
{
if((Options & int(StandardCompressedFormat)) && (!expr.isCompressed()))
{
m_object = expr;
Base::construct(m_object);
TPlainObjectType* obj = reinterpret_cast<TPlainObjectType*>(m_object_bytes);
::new (obj) TPlainObjectType(expr);
Base::construct(*obj);
}
else
{
@ -194,12 +195,13 @@ class Ref<const SparseMatrix<MatScalar,MatOptions,MatIndex>, Options, StrideType
template<typename Expression>
void construct(const Expression& expr, internal::false_type)
{
m_object = expr;
Base::construct(m_object);
TPlainObjectType* obj = reinterpret_cast<TPlainObjectType*>(m_object_bytes);
::new (obj) TPlainObjectType(expr);
Base::construct(*obj);
}
protected:
TPlainObjectType m_object;
char m_object_bytes[sizeof(TPlainObjectType)];
};