Re: [FFmpeg-devel] [GASPP PATCH] Implicitly start out in the text section for armasm

[Henrik Gramner via ffmpeg-devel]

On Wed, Apr 3, 2024 at 3:47 PM Martin Storsjö  wrote:
>
> This fixes assembling files starting with bare symbol declarations,
> without explicitly switching to .text first.

lgtm.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [GASPP PATCH] Implicitly start out in the text section for armasm

[Martin Storsjö]

This fixes assembling files starting with bare symbol declarations,
without explicitly switching to .text first.
---
 gas-preprocessor.pl | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
index 2880858..b66181a 100755
--- a/gas-preprocessor.pl
+++ b/gas-preprocessor.pl
@@ -289,6 +289,9 @@ my %aarch64_req_alias;
 if ($force_thumb) {
 parse_line(".thumb\n");
 }
+if ($as_type eq "armasm") {
+parse_line(".text\n");
+}
 
 # pass 1: parse .macro
 # note that the handling of arguments is probably overly permissive vs. gas
-- 
2.34.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".