Will do— I think I now have a good patch set, but I want to check it
works.  I'm currently doing a fresh build.

One other question Mans:

Because the baseline architecture used by the toolchain is
-march=armv5t, many of the NEON sources fail to build even with
-mfpu=neon, because of the presence of PLD instructions (not supported
in v5).

My current fix for this is to add -march=armv7-a in extra-cflags for the
whole NEON-enabled build pass: my rationale for this is that the
presence of NEON implies v7, which means that v7 features may as well be
enabled for the whole pass. (This seems a general problem with -mfpu=
and -march/cpu= which allow non-existent architectures, such as v5+NEON
or NEON-without-v7 to be selected when used together.)

Is this the right thing to do, or do you have a different suggestion?

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to