[MPlayer-dev-eng] [PATCH] fix compilation on intel mac
Nicolas Plourde
nicolas.plourde at gmail.com
Tue Apr 4 23:49:06 CEST 2006
On 4-Apr-06, at 5:45 PM, Rich Felker wrote:
> On Wed, Mar 22, 2006 at 07:58:33AM -0500, Nicolas Plourde wrote:
>> +#ifdef MACOSX
>> + ".align 4\n\t"
>> +#elif
>> ".balign 16\n\t"
>> +#endif
>> "1: \n\t"
>> "movaps (%0, %%"REG_S"), %%xmm0 \n\t"
>> "movaps 16(%0, %%"REG_S"), %%xmm1\n\t"
>> @@ -714,7 +718,11 @@
>> "movlps %1, %%xmm7 \n\t"
>> "shufps $0x00, %%xmm7, %%xmm7 \n\t"
>> "mov $-1024, %%"REG_S" \n\t"
>> +#ifdef MACOSX
>> + ".align 4\n\t"
>> +#elif
>> ".balign 16\n\t"
>> +#endif
>
> Repeating this nonsense over and over is absolutely incorrect. I'm
> slightly against the patch in general, but if you want to do something
> like this, you must:
>
> 1. Test in configure to see if the assembler supports .balign.
> 2. #define a macro ALIGN_16_ASM to ".align 4" or ".balign 16"
> depending on the results of the test from 1.
> 3. Replace all occurrances of ".balign 16" with ALIGN_16_ASM.
>
> Both OS-specific ifdefs (rather than feature-specific ones) and
> repeated redundant ifdefs are disgusting and unacceptable from a code
> maintainence standpoint.
>
There is no ifdef in the patch i sent this morning. I use #define
ASMALIGN16. Dont take into account the ffmpeg patch its not inteded
to be applied has is. Im attaching the mplayer specific patch again
for review.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: intel_osx.diff
Type: application/octet-stream
Size: 30675 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060404/b085f1c9/attachment.obj>
-------------- next part --------------
====
Nicolas Plourde <nicolas.plourde at gmail.com>
More information about the MPlayer-dev-eng
mailing list