[ODE] Patch to optimize contactGetInfo2 submitted on SF
remi.ricard at simlog.com
Wed Jul 12 12:24:37 MST 2006
> On 7/11/06, Remi Ricard <remi.ricard at simlog.com> wrote:
>> This patch do some simple optimization of the function
>> - Unroll for loops and delete the indexing variable
>> - Remove unneeded initialization
>> - Do a check only when needed. ( move the "if (j->the_m >= 3)" inside
>> the "if (j->the_m >= 2)"
> I unrolled the loops, but I did not make the other changes as I felt
> they reduced readability for no real performance gain.
It was easier to read or understand than the LCP ;-)
Leaving the if outside should not make a big difference but small
differences add up.
> If someone can
> demonstrate a measurable performance gain by removing the one
> conditional then I'll move it.
My evaluation of vtune is over so I can't do the check....
> Thanks for the patch!
No problem, thanks for applying the patch.
More information about the ODE