> Am 20.07.2026 um 18:42 schrieb Jakub Jelinek <[email protected]>:
> 
> 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?

Ok

Richard 

> 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
> 

Reply via email to