octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #8093] Convert isscalar to a built-in func


From: Jason Alan Palmer
Subject: [Octave-patch-tracker] [patch #8093] Convert isscalar to a built-in function
Date: Wed, 26 Jun 2013 00:53:01 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20100101 Firefox/17.0

Follow-up Comment #3, patch #8093 (project octave):

I wasn't aware that C++ functions are frowned upon. I don't have any
particular case where I'd call isscalar a bottleneck, just some scripts where
it is heavily used and gets pretty high on my profile results:

Function          Time (s)    Calls
-----------------------------------
some_function     133.64    1075200
another_function  133.219   1075200
isscalar          119.322  12906470
more_function     114.951   1075200
my_function       107.563       800
.
.
.
(I realize it got up there because it was called over ten million times....)

A quick check shows the built-in to be about 4x as fast on my machine.

(file #28421)
    _______________________________________________________

Additional Item Attachment:

File name: isscalar_built-in.diff         Size:3 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8093>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]