On 2014-01-21 09:35:19 +0200, Martin Storsjö wrote:
> On Mon, 20 Jan 2014, Janne Grunau wrote:
> 
> > Handles fft4_neon and fft65536_neon from libavcodec/arm/fft_neon.S
> > correctly which are only called through pc-relative offsets. Fixes
> > i?mdct-4 fate tests on iOS with thumb.
> >
> > Signed-off-by: Janne Grunau <janne-li...@jannau.net>
> > ---
> > gas-preprocessor.pl | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
> > index 2072ca3..cdfb95d 100755
> > --- a/gas-preprocessor.pl
> > +++ b/gas-preprocessor.pl
> > @@ -483,6 +483,12 @@ foreach my $line (@pass1_lines) {
> >         }
> >     }
> >
> > +    if ($thumb and $line =~ /^\s*\.word\s+(\w+)\s*$/) {
> > +        if (exists $thumb_labels{$1}) {
> > +            print ASMFILE ".thumb_func $1\n";
> > +        }
> > +    }
> > +
> >     # @l -> lo16()  @ha -> ha16()
> >     $line =~ s/,\s+([^,]+)\@l\b/, lo16($1)/g;
> >     $line =~ s/,\s+([^,]+)\@ha\b/, ha16($1)/g;
> > -- 
> > 1.8.5.3
> 
> Ok

Doesn't work unfortunately, forgot --enable-pic in test build

Breaks in vp8_armv6.S due to a '.thumb_func 0'. I'm thinking of
replacing the .func directive with .thumb_func instead of commenting it
out. That makes at least sure that all functions are locally declared as
.thumb_func

Janne
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to