[libcxx-commits] [libcxx] [libc++] Fix naming of benchmarks with new SPEC metrics (PR #178445)

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jan 28 07:25:53 PST 2026


https://github.com/ldionne created https://github.com/llvm/llvm-project/pull/178445

We need to ensure that no dots end up in those names to avoid confusing LNT and the compare-benchmarks script.

>From 8fce1a5d0077d4d5c2e264a0a1403cdb8fc8c70b Mon Sep 17 00:00:00 2001
From: Louis Dionne <ldionne.2 at gmail.com>
Date: Wed, 28 Jan 2026 10:24:20 -0500
Subject: [PATCH] [libc++] Fix naming of benchmarks with new SPEC metrics

We need to ensure that no dots end up in those names to avoid
confusing LNT and the compare-benchmarks script.
---
 libcxx/test/benchmarks/spec.gen.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libcxx/test/benchmarks/spec.gen.py b/libcxx/test/benchmarks/spec.gen.py
index c08541a57ac7c..41d8032babc3a 100644
--- a/libcxx/test/benchmarks/spec.gen.py
+++ b/libcxx/test/benchmarks/spec.gen.py
@@ -88,5 +88,5 @@
 
     # If there were no errors, parse the SPEC results and the `time` output into LNT-compatible format and print them.
     print(f'RUN: %{{libcxx-dir}}/utils/parse-spec-results %{{temp}}/result/*.train.csv --output-format=lnt > %{{temp}}/results.lnt')
-    print(f'RUN: %{{libcxx-dir}}/utils/parse-time-output %{{temp}}/time.txt --benchmark {benchmark} --extract instructions max_rss cycles peak_memory >> %{{temp}}/results.lnt')
+    print(f'RUN: %{{libcxx-dir}}/utils/parse-time-output %{{temp}}/time.txt --benchmark {benchmark.replace(".", "_")} --extract instructions max_rss cycles peak_memory >> %{{temp}}/results.lnt')
     print(f'RUN: cat %{{temp}}/results.lnt')



More information about the libcxx-commits mailing list