The change looks good to me.

-----Original Message-----
From: Andi Kleen <[email protected]> 
Sent: Thursday, June 25, 2026 1:32 PM
To: [email protected]
Cc: [email protected]; Eugene Rozenfeld <[email protected]>; Andi 
Kleen <[email protected]>
Subject: [EXTERNAL] [PATCH] Support overriding of CREATE_GCOV for test suite

From: Andi Kleen <[email protected]>

create_gcov use is currently hard coded in the test suite. Allow to override it 
with a environment variable similar to what the autoprofiled bootstrap allows.

gcc/testsuite/ChangeLog:

        * lib/profopt.exp (profopt-execute): Check if CREATE_GCOV env
        variable exists and use it when there.

Signed-off-by: Andi Kleen <[email protected]>
---
 gcc/testsuite/lib/profopt.exp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/gcc/testsuite/lib/profopt.exp b/gcc/testsuite/lib/profopt.exp 
index 370b45436fb8..4f3f61958aa5 100644
--- a/gcc/testsuite/lib/profopt.exp
+++ b/gcc/testsuite/lib/profopt.exp
@@ -462,7 +462,11 @@ proc profopt-execute { src } {
                } else {
                    set profiler "perf"
                }
-               set cmd "create_gcov --binary $execname1 
--profile=$tmpdir/$base.perf.data --profiler=$profiler 
--gcov_version=$autofdo_version --gcov=$tmpdir/$bprefix$base.$ext"
+               set create_gcov "[getenv CREATE_GCOV]"
+               if { $create_gcov == "" } {
+                   set create_gcov "create_gcov"
+               }
+               set cmd "$create_gcov --binary $execname1 
--profile=$tmpdir/$base.perf.data --profiler=$profiler 
--gcov_version=$autofdo_version --gcov=$tmpdir/$bprefix$base.$ext"
                verbose "Running $cmd"
                set id [remote_spawn "" $cmd]
                if { $id < 0 } {
--
2.54.0

Reply via email to