On Tue, 14 Jan 2014, Janne Grunau wrote:
Xcode 5 and 5.1 complain over the missing shift although it's optional according to the reference manual. --- gas-preprocessor.pl | 4 ++++ 1 file changed, 4 insertions(+)diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl index 48cd6c0..a0287a2 100755 --- a/gas-preprocessor.pl +++ b/gas-preprocessor.pl @@ -625,6 +625,10 @@ sub handle_serialized_line { if ($line =~ /^\s*mov\s+(v\d[\.{}\[\]\w]+),\s*(v\d[\.{}\[\]\w]+)\b\s*$/) { $line = " orr $1, $2, $2\n"; } + # movi 8, 16, 32 bit shifted variant, shift is optional + if ($line =~ /^\s*movi\s+(v[0-3]?\d\.(?:2|4|8|16)[bhsBHS])\s*,\s*(#\d+)\b\s*$/) { + $line = " movi $1, $2, lsl #0\n"; + } }
Does the assembler support the immediate constant as a hex literal? In that case I guess the regexp could be extended to handle that as well. Other than that it looks good, but the same remark about the indentation as on the previous patch.
// Martin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
