On 2014-01-14 19:45:40 +0200, Martin Storsjö wrote:
> On Tue, 14 Jan 2014, Janne Grunau wrote:
>
> > Clang from Xcode5 supports bsl/saddl2? only in an unofficial assembler
> > syntax. To avoid breakage with later versions or different assembler
> > gas-preprocessor.pl checkss for the enviroment variable GASPP_FIX_XCODE5
> > before changing these instructions to the non-standard syntax.
> > ---
> > gas-preprocessor.pl | 10 ++++++++++
> > 1 file changed, 10 insertions(+)
> >
> > diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
> > index cdce3fa..5e68b30 100755
> > --- a/gas-preprocessor.pl
> > +++ b/gas-preprocessor.pl
> > @@ -633,6 +633,16 @@ sub handle_serialized_line {
> > if ($line =~
> > /^\s*uxtl(2)?\s+(v[0-3]?\d\.[248][hsdHSD])\s*,\s*(v[0-3]?\d\.(?:4|8|16)[bhsBHS])\b\s*$/)
> > {
> > $line = " ushll$1 $2, $3, #0\n";
> > }
> > + if ($ENV{GASPP_FIX_XCODE5}) {
> > + if ($line =~ /^\s*bsl\b/) {
> > + $line =~ s/\b(bsl)(\s+v\d+\.(\w+))\b/$1.$3$2/;
> > + $line =~ s/\b(v\d+)\.$3\b/$1/g;
> > + }
> > + if ($line =~ /^\s*saddl2?\b/) {
> > + $line =~ s/\b(saddl2?)(\s+v\d+\.(\w+))\b/$1.$3$2/;
> > + $line =~ s/\b(v[0-3]?\d)\.\w+\b/$1/g;
> > + }
> > + }
> > }
> >
> > print ASMFILE $line;
> > --
> > 1.8.5.2
>
> Ok except for the indentation.
>
> Btw, why do you write v[0-3]?\d in some places and v\d+ in other places?
no specific reason, changed this patch to consistently use v[0-3]?\d
Janne
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
