[clang] 9db72e5 - [clang][cmake] Fix support for dynamic libraries in CLANG_BOLT

via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 3 00:21:22 PST 2025


Author: serge-sans-paille
Date: 2025-03-03T09:21:05+01:00
New Revision: 9db72e55edf8c10b2a1b72f1a2d4594d312dd91c

URL: https://github.com/llvm/llvm-project/commit/9db72e55edf8c10b2a1b72f1a2d4594d312dd91c
DIFF: https://github.com/llvm/llvm-project/commit/9db72e55edf8c10b2a1b72f1a2d4594d312dd91c.diff

LOG: [clang][cmake] Fix support for dynamic libraries in CLANG_BOLT

Patch typo introduced in #127020

Added: 
    

Modified: 
    clang/utils/perf-training/perf-helper.py

Removed: 
    


################################################################################
diff  --git a/clang/utils/perf-training/perf-helper.py b/clang/utils/perf-training/perf-helper.py
index cdb6c39f6c50e..7a049e71d9826 100644
--- a/clang/utils/perf-training/perf-helper.py
+++ b/clang/utils/perf-training/perf-helper.py
@@ -560,12 +560,12 @@ def genOrderFile(args):
     return 0
 
 
-def filter_bolt_optimized(inputs, instrumented_outputs):
+def filter_bolt_optimized(inputs, instrumented_outputs, readelf):
     new_inputs = []
     new_instrumented_ouputs = []
     for input, instrumented_output in zip(inputs, instrumented_outputs):
         output = subprocess.check_output(
-            [opts.readelf, "-WS", input], universal_newlines=True
+            [readelf, "-WS", input], universal_newlines=True
         )
 
         # This binary has already been bolt-optimized, so skip further processing.
@@ -597,7 +597,9 @@ def bolt_optimize(args):
         instrumented_outputs
     ), "inconsistent --input / --instrumented-output arguments"
 
-    inputs, instrumented_outputs = filter_bolt_optimized(inputs, instrumented_outputs)
+    inputs, instrumented_outputs = filter_bolt_optimized(inputs,
+                                                         instrumented_outputs,
+                                                         opts.readelf)
     if not inputs:
         return 0
 


        


More information about the cfe-commits mailing list