We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 929 - Remove MMX mixers?
Summary: Remove MMX mixers?
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: audio (show other bugs)
Version: HG 2.0
Hardware: Other All
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-09 23:48 UTC by Ryan C. Gordon
Modified: 2012-01-08 00:09 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan C. Gordon 2010-01-09 23:48:51 UTC 
We disabled the MMX mixers for 1.2.14. They produce garbage audio.

I'm inclined to remove the code outright for 1.3. Surely we should not be targeting MMX at this point anyhow.

Grep for SDL_BUGGY_MMX_MIXERS in 1.3.

--ryan.
Comment 1 Sam Lantinga 2010-01-10 01:34:48 UTC 
Ryan:  The compiler probably generates better SSE code from the C code at this point.
 Sam:  Ah, true
And I haven't timed it, but I imagine on modern CPUs the hit is from memory access rather than math
Comment 2 Sam Lantinga 2011-01-20 18:10:22 UTC 
Ryan's going to remove these.
Comment 3 Sam Lantinga 2012-01-08 00:09:35 UTC 
This was done.