On 15/07/2026 03:14, Richard Biener wrote:
I'm just looking for a simple solution here, not reengineer things with exciting C++ ok?The gen-pass-instances.awk change looks OK. I find the Makefile change just obfuscates things, I can no longer see what happens, so NACK from my side for that part.
Fair enough, I committed this nathan -- Nathan Sidwell
From f683617744063c0032ae046f22346c92768dcba3 Mon Sep 17 00:00:00 2001 From: Nathan Sidwell <[email protected]> Date: Mon, 13 Jul 2026 09:21:45 -0400 Subject: [PATCH] Robustify awk pass generation gcc/ * gen-pass-instances.awk: Require only white-space before macro invocations. --- gcc/gen-pass-instances.awk | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gcc/gen-pass-instances.awk b/gcc/gen-pass-instances.awk index 66484e8c563..300e3ebad5b 100644 --- a/gcc/gen-pass-instances.awk +++ b/gcc/gen-pass-instances.awk @@ -167,17 +167,17 @@ function replace_pass(line, fnname, num, i) lines[num] = new_line; } -/INSERT_PASS_AFTER \(.+\)/ { +/^[[:blank:]]*INSERT_PASS_AFTER \(.+\)/ { insert_pass($0, "INSERT_PASS_AFTER", 1); next; } -/INSERT_PASS_BEFORE \(.+\)/ { +/^[[:blank:]]*INSERT_PASS_BEFORE \(.+\)/ { insert_pass($0, "INSERT_PASS_BEFORE", 0); next; } -/REPLACE_PASS \(.+\)/ { +/^[[:blank:]]*REPLACE_PASS \(.+\)/ { replace_pass($0, "REPLACE_PASS"); next; } -- 2.55.0
