[quake3-bugzilla] [Bug 3639] BoxOnPlaneSide patch

[bugzilla-daemon at icculus.org]

http://bugzilla.icculus.org/show_bug.cgi?id=3639

Daniel Gibson <metalcaedes at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |metalcaedes at gmail.com

--- Comment #10 from Daniel Gibson <metalcaedes at gmail.com> 2009-09-18 22:19:33 EDT ---
I modified Patricks Benchmark code to use the assembly code. On my notebook
(Pentium m 1,6GHz throttled to 800MHz, 2GB RAM, debian Lenny with stock kernel)
I got the following benchmark results (NR_TESTS = 1M, NR_REPEAT = 1024):
new C: avg of 68ms, 69ms, 67ms (3 test runs)
old ASM: avg of 76ms, 76ms, 76ms (3 test runs)
old C: avg of 84ms, 83ms, 84ms (3 test runs)

At least on my machine the new C code is faster than the old ASM code.

I also checked the correctness of the code (compared each case of the
switch-case-statement in the old code with what would happen in the new code)
and I'm quite certain that the proposed implementation is correct.
I attached a text file with some comments to the non-trivial code-snippets of
each implementation. 

Cheers,
Daniel

-- 
Configure bugmail: http://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.