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 4791

Summary: Correct vec_perm() application on little-endian 64-bit PowerPC (SDL-1.2)
Product: SDL Reporter: Ozkan Sezer <sezeroz>
Component: videoAssignee: Sam Lantinga <slouken>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2    
Version: HG 1.2   
Hardware: PowerPC   
OS: Linux   
Attachments: ppc64le altivec patch
ppc64le altivec patch

Description Ozkan Sezer 2019-09-06 10:47:03 UTC 
Created attachment 3954 [details]
ppc64le altivec patch

The attached patch is from RedHat/Fedora, details of the issue and
the analysis / discussion of it are here:
https://bugzilla.redhat.com/show_bug.cgi?id=1392465

The patch has been applied in RedHat / Fedora since two years.  If
there are no objections, I will apply the it to the SDL-1.2 branch
in a few days.
Comment 1 Ozkan Sezer 2019-09-06 11:16:34 UTC 
Created attachment 3955 [details]
ppc64le altivec patch

Sorry, the patch was generated against 2.0:
The correct version is attached. Awaiting objections
(and/or approval.)

Surprisingly, the altivec code paths seem similar to
1.2 ones in the 2.0 tree, but noone has yet complained
about ppc64le...
Comment 2 Sam Lantinga 2019-09-06 15:51:09 UTC 
Patches added, thanks!
https://hg.libsdl.org/SDL/rev/9aa7aea27dec
https://hg.libsdl.org/SDL/rev/5df049ec4620
Comment 3 Ozkan Sezer 2019-09-06 16:27:38 UTC 
I didn't actually intend that this go into 2.0. Let's see if anyone screams.