Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct vec_perm() application on little-endian 64-bit PowerPC (SDL-1.2) #807

Closed
SDLBugzilla opened this issue Feb 10, 2021 · 0 comments
Closed

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 1.2
Reported for operating system, platform: Linux, PowerPC

Comments on the original bug report:

On 2019-09-06 10:47:03 +0000, Ozkan Sezer wrote:

Created attachment 3954
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.

On 2019-09-06 11:16:34 +0000, Ozkan Sezer wrote:

Created attachment 3955
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...

On 2019-09-06 15:51:09 +0000, Sam Lantinga wrote:

Patches added, thanks!
https://hg.libsdl.org/SDL/rev/9aa7aea27dec
https://hg.libsdl.org/SDL/rev/5df049ec4620

On 2019-09-06 16:27:38 +0000, Ozkan Sezer wrote:

I didn't actually intend that this go into 2.0. Let's see if anyone screams.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant