[vlc-devel] [PATCH] arm_neon: updated syntax to suit the more strict current clang releases

[Måns Rullgård]

"Rémi Denis-Courmont" <remi at remlab.net> writes:

> Le mercredi 1 août 2012 13:41:22 Måns Rullgård, vous avez écrit :
>> Felix Paul Kühne <fkuehne.videolan at gmail.com> writes:
>> > Hey guys,
>> > 
>> > the attached patch updates some of the assembly syntax for ARM to suit
>> > the latest clang releases, which are a tad more strict about the
>> > syntax. A similar issue was discovered and fixed in libav's NEON code
>> > already.
>> 
>> Clang is not the problem, Apple's assembler is.
>
> What's the problem? Is the code conforming to divided syntax, or is it not?

Divided?

The code uses the old-style mnemonics for a few instructions, and
apparently those are no longer accepted by Apple's assembler.  Switching
to the new (unified) syntax is the right thing to do, and (some versions
of) the GNU assembler need to be switched to that mode with the .syntax
directive.

-- 
Måns Rullgård
mans at mansr.com