[vlc-devel] [PATCH] contrib: Fix ARMv6 compilation

[Måns Rullgård]

Edward Wang <edward.c.wang at compdigitec.com> writes:

> On Tuesday, June 12, 2012, Måns Rullgård <mans at mansr.com> wrote:
>> Edward Wang <edward.c.wang at compdigitec.com> writes:
>>
>>> The contrib flags got messed up in
>>> d61052a0987d751ecda0b763c900f834f3358612. This led to VFP instructions
>>> being used on non-NEON devices,
>>
>> Most ARMv6 devices have VFP, so that should not be a problem.  Which
>> exact chip are you using?
> The same one I used for my Android VLC codec benchmarks:
>
> Motorola XT300 - Android 2.1, 500 Mhz, 256 MB, rooted with busybox,
> Qualcomm MSM7225 (ARM1136EJ-S core, ARMv6TEJ architecture, no NEON
> extensions)

There is no such thing as an ARM1136EJ-S.  There is only ARM1136J-S and
ARM1136JF-S, so which is it?  There is also no such thing as ARMv6TEJ
architecture.  I suggest you review your choice of information sources
regarding the ARM architecture and implementations.

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