Re: [PATCH] passes: Make gcc buildable again with mawk <= 1.3.3

Thanks Jakub!

On 20/07/2026 12:41, Jakub Jelinek wrote:
> On Wed, Jul 15, 2026 at 12:49:50PM -0400, Nathan Sidwell wrote:
>> Fair enough, I committed this
> 
> Unfortunately it seems some versions of Debian/Ubuntu use mawk instead
> of gawk by default, and mawk pretends to be POSIX compatible, but at least
> versions <= 1.3.3 didn't support POSIX character classes.
> We don't really need the various UTF-8 fancy blanks, and even this very
> own script already uses sub(/^[ \t]*/, "", arg3); elsewhere, so this
> patch just replaces [[:blank:]] with [ \t].
> 
> Tested on x86_64-linux (though, Fedora with gawk), ok for trunk?
> 
> 2026-07-20  Jakub Jelinek  <[email protected]>
> 
> 	* gen-pass-instances.awk: Use [ \t] instead of [[:blank:]]
> 	for compatibility with mawk <= 1.3.3.
> 
> --- gcc/gen-pass-instances.awk.jj	2026-07-16 09:44:33.627721893 +0200
> +++ gcc/gen-pass-instances.awk	2026-07-20 18:32:00.542130598 +0200
> @@ -167,17 +167,17 @@ function replace_pass(line, fnname,			nu
>     lines[num] = new_line;
>   }
>   
> -/^[[:blank:]]*INSERT_PASS_AFTER \(.+\)/ {
> +/^[ \t]*INSERT_PASS_AFTER \(.+\)/ {
>     insert_pass($0, "INSERT_PASS_AFTER", 1);
>     next;
>   }
>   
> -/^[[:blank:]]*INSERT_PASS_BEFORE \(.+\)/ {
> +/^[ \t]*INSERT_PASS_BEFORE \(.+\)/ {
>     insert_pass($0, "INSERT_PASS_BEFORE", 0);
>     next;
>   }
>   
> -/^[[:blank:]]*REPLACE_PASS \(.+\)/ {
> +/^[ \t]*REPLACE_PASS \(.+\)/ {
>     replace_pass($0, "REPLACE_PASS");
>     next;
>   }
> 
> 
> 	Jakub
> 


-- 
Nathan Sidwell