[Octave-bug-tracker] [bug #56752] Performance slowdown from version 3.2.4 through to current dev branch

[Rik]

Follow-up Comment #20, bug #56752 (project octave):

@jwe: I tested the preliminary patch for octave_value_list and it looks good. 
My benchmarking shows that it takes the runtime from 13.8 seconds to 11.7
seconds, or -21%.  I'm happy to say that I roughly nailed it when I said
changing to std::vector might bring about a 25% savings.  Still, if we could
figure out how to reduce the number of calls to octave_value_list for ordinary
evaluations that would help even more.

This looks so promising that I would immediately polish it up and commit it. 
Since you ran 'make check', I assume this didn't cause any regressions.

Once in Mercurial, I want to try a mini-optimization.  My guess is that the
size of octave_value_list objects is almost always 0 or 1.  If we change the
constructor to always reserve capacity for at least 1 object, do we get
another small speed boost?



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?56752>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/