On 2011-08-03, Donovan Watteau <[email protected]> wrote:
> Sorry to bump this again, but I think it deserves to make the release
> since the bug is important and the fix is easy (a similar one is already
> used to avoid using SSE2)...

I think the patch for SSE2 is not actually doing anything useful and should
be removed, SSE2 is not defined on i386 and AIUI amd64-capable CPUs support
SSE2 anyway.

>>> I see there this a patch in the port [1] which undef's __SSE2__ so I
>>> guess the same thing should be done for __ALTIVEC__. Unfortunately,
>>> it wasn't possible for me to test the patch, since it would take forever
>>> to compile on such an old box. But I guess the issue is here.
>>>
>>> I'd gladly test any new package built with this patch on my G3, if it
>>> helps.

Complete tested diffs have a much better chance of getting accepted.
Yes it takes a while but it will for anyone else who might look at it
and you actually have the machine to test it on. It's only going to be
a bit slower on your machine than the sort of macppc which most
developers have access to...


Reply via email to