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 - Correct vec_perm() application on little-endian 64-bit PowerPC (SDL-1.2)
Summary: Correct vec_perm() application on little-endian 64-bit PowerPC (SDL-1.2)
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: video (show other bugs)
Version: HG 1.2
Hardware: PowerPC Linux
: P2 normal
Assignee: Sam Lantinga
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-06 10:47 UTC by Ozkan Sezer
Modified: 2019-09-06 16:27 UTC (History)
0 users

See Also:

Attachments
ppc64le altivec patch (4.56 KB, patch)
2019-09-06 10:47 UTC, Ozkan Sezer 		Details | Diff
ppc64le altivec patch (4.65 KB, patch)
2019-09-06 11:16 UTC, Ozkan Sezer 		Details | Diff
Show Obsolete (1) View All 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 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.