eessppeelllloo/ffmpeg
1
0
Fork 0
mirror of https://git.ffmpeg.org/ffmpeg.git synced 2026-06-16 04:32:47 +02:00
Code Issues Packages Projects Releases Wiki Activity

tools/indent_arm_assembly: Recognize more comment forms, for skipping lowercasing

Browse Source 
When we try to lowercase register names (e.g. Q0 -> q0) we avoid
doing that for parts of the code that are comments, as comments
occasionally contain pseudocode that contain such mentions that
aren't register names, but pseudocode/reference code variables.
See 7ebb6c54eb for more details
about that.

In addition to recognizing comments starting with //, also
recognize /* and @ (which is a comment char in arm assembly, but
not in aarch64).
This commit is contained in:
Martin Storsjö
2026-04-10 12:25:56 +03:00
parent 7c67748537
commit cd7a3cd799
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/indent_arm_assembly.pl
+3 -3
View File
@@ -221,12 +221,12 @@ while (<$in>) {
# Lowercase register names. Only apply this on lines up to
# comments, as this can match common spec/code references in
# code comments. Split the string on // for comments, apply the
# code comments. Split the string on //, /* or @ for comments, apply the
# substitution on the first segment (up to a comment char), and
# join the string again.
my @parts = split(/\/\//, $rest);
my @parts = split(/(\/\/|\/\*|@)/, $rest);
$parts[0] =~ s/\b([XWVQDSHBZP][0-9]+)\b/lc($1)/ge;
$rest = join('//', @parts);
$rest = join('', @parts);
# Lowercase the aarch64 vector layout description, .8B -> .8b
$rest =~ s/(\.[84216]*[BHSD])/lc($1)/ge;