[libav-devel] [GASPP PATCH] Use the correct variable $line instead of the implicit variable

2018-10-22 Thread Martin Storsjo
This fixes cases if the input parameter is something else than the currently iterated variable. --- gas-preprocessor.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl index 41d7b69..39ad08d 100755 --- a/gas-preprocessor.pl +++ b/ga

[libav-devel] [GASPP PATCH] Add a -verbose option for printing all executed commands

2018-10-22 Thread Martin Storsjo
--- gas-preprocessor.pl | 9 + 1 file changed, 9 insertions(+) diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl index fd9aac8..41d7b69 100755 --- a/gas-preprocessor.pl +++ b/gas-preprocessor.pl @@ -27,6 +27,7 @@ my $as_type = "apple-gas"; my $fix_unreq = $^O eq "darwin"; my $for

Re: [libav-devel] [GASPP PATCH 2/2] Don't match whitespace as branch condition codes

2018-10-22 Thread Janne Grunau
On 2018-10-20 00:18:27 +0300, Martin Storsjö wrote: > For cases like "b1b", this could previously be matched as > $cond = " ". > > This fixes preprocessing with a preprocessor that preserves multiple > consecutive spaces, like cl.exe does. > --- > Better fix, which also works in a number of c

Re: [libav-devel] [GASPP PATCH] Extend armasm64 workaround for uxtw/sxtw to uxth/sxth and uxtb/sxtb as well

2018-10-22 Thread Janne Grunau
On 2018-10-22 12:51:47 +0300, Martin Storsjö wrote: > --- > gas-preprocessor.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl > index 7efe3b9..669d435 100755 > --- a/gas-preprocessor.pl > +++ b/gas-preprocessor.pl > @@ -1011,7 +

Re: [libav-devel] [GASPP PATCH 1/2] Fix the regexp used for replacing .align

2018-10-22 Thread Luca Barbato
On 19/10/2018 23:07, Martin Storsjo wrote: > The condition above allows multiple spaces, but the actual replacement > only allowed one space. > --- > gas-preprocessor.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl > index c563

Re: [libav-devel] [GASPP PATCH] Extend armasm64 workaround for uxtw/sxtw to uxth/sxth and uxtb/sxtb as well

2018-10-22 Thread Luca Barbato
On 22/10/2018 11:51, Martin Storsjo wrote: > --- > gas-preprocessor.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl > index 7efe3b9..669d435 100755 > --- a/gas-preprocessor.pl > +++ b/gas-preprocessor.pl > @@ -1011,7 +1011,7 @@

Re: [libav-devel] [GASPP PATCH 2/2] Don't match whitespace as branch condition codes

2018-10-22 Thread Luca Barbato
On 19/10/2018 23:18, Martin Storsjo wrote: > For cases like "b1b", this could previously be matched as > $cond = " ". > > This fixes preprocessing with a preprocessor that preserves multiple > consecutive spaces, like cl.exe does. > --- > Better fix, which also works in a number of cases wher

[libav-devel] [GASPP PATCH] Extend armasm64 workaround for uxtw/sxtw to uxth/sxth and uxtb/sxtb as well

2018-10-22 Thread Martin Storsjo
--- gas-preprocessor.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl index 7efe3b9..669d435 100755 --- a/gas-preprocessor.pl +++ b/gas-preprocessor.pl @@ -1011,7 +1011,7 @@ sub handle_serialized_line { # Convert e.g. "