Tried noalias() for collect(), but no real improvement

release/4.3a0
Alex Cunningham 2010-01-20 03:25:55 +00:00
parent 3bf15333af
commit 55e414cbdd
2 changed files with 4 additions and 3 deletions

View File

@ -513,7 +513,7 @@ Matrix collect(const std::vector<const Matrix *>& matrices, size_t m, size_t n)
// BOOST_FOREACH(const Matrix* M, matrices) {
// ublas::matrix_range<Matrix> mr(A, ublas::range(0, dimA1),
// ublas::range(hindex, hindex+M->size2()));
// mr = *M;
// noalias(mr) = *M;
// hindex += M->size2();
// }

View File

@ -14,8 +14,9 @@ using namespace gtsam;
/*
* Results:
* Alex's Machine:
* - (1st pass of simple changes) no pass: 0.184 sec , pass: 0.181 sec
* - (1st rev memcpy) no pass: 0.181 sec , pass: 0.180 sec
* - (1st pass of simple changes) no pass: 0.184 sec , pass: 0.181 sec
* - (1st rev memcpy) no pass: 0.181 sec , pass: 0.180 sec
* - (1st rev matrix_range) no pass: 0.186 sec , pass: 0.184 sec
*/
double timeCollect(size_t p, size_t m, size_t n, bool passDims, size_t reps) {
// create a large number of matrices