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 2241

Summary: SSE intrinsic in fillrect MMX path
Product: SDL Reporter: norfanin
Component: videoAssignee: Sam Lantinga <slouken>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2    
Version: HG 2.0   
Hardware: All   
OS: All   

Description norfanin 2013-11-14 17:05:01 UTC 
The MMX path in SDL_fillrect.c uses the SSE intrinsic _mm_stream_pi. The function or symbol provided by the compiler will not be present because the SSE header may not get included. The linker will complain about an undefined reference.

Since this is the only intrinsic used here (and someone forgot to create one for MOVQ), I think the MMX path can be removed completely. At least I don't see another way to move 64-bits from an MMX register to memory.
Comment 1 Sam Lantinga 2013-11-16 19:56:07 UTC 
Good point. Since most CPUs these days support SSE, an MMX-only codepath isn't very useful, so I'm going to pull it.

Thanks!
https://hg.libsdl.org/SDL/rev/097e3fac1979