[FFmpeg-devel] [PATCH 1/6] truehd: add hand-scheduled ARM asm version of mlp_filter_channel.

Michael Niedermayer michaelni at gmx.at
Wed Mar 19 20:14:19 CET 2014


On Wed, Mar 19, 2014 at 06:23:42PM -0000, Ben Avison wrote:
> On Wed, 19 Mar 2014 17:50:33 -0000, Michael Niedermayer <michaelni at gmx.at> wrote:
> >>+// This code uses too many ARM-only tricks to easily assemble as Thumb
> >>+.arm
> >>+#undef CONFIG_THUMB
> >>+#define CONFIG_THUMB 0
> >
> >this results in a warning when config.h is included later
> [...]
> >also compilation fails with:
> 
> Ah, that's unfortunate. I thought I'd neatly sidestepped the ARM/Thumb
> differences there - as you saw, I noticed that I'd used a *lot* of
> instructions that Thumb doesn't support.
> 
> I suppose I could change libavutil/arm/asm.S to accept an override for
> the CONFIG_THUMB setting from config.h, but as this is a one-off, perhaps
> a slightly different fudge will do... would you mind trying this to see
> if it works please?

it builds but segfaults at runtime in
make fate-lossless-meridianaudio

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140319/b533d202/attachment.asc>


More information about the ffmpeg-devel mailing list