[llvm] [test] Make `xxhash` benchmark target name less generic (PR #100172)

Daniel Bertalan via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 23 10:57:03 PDT 2024


https://github.com/BertalanD created https://github.com/llvm/llvm-project/pull/100172

Downstream users reported that the `xxhash` target name conflicts with some of their existing build targets. Rename it to `bench-xxhash`.

See https://github.com/llvm/llvm-project/pull/99634#discussion_r1687666875

>From ae73c7d2b301d0c670bd40b61c2b7df4d151cfe6 Mon Sep 17 00:00:00 2001
From: Daniel Bertalan <dani at danielbertalan.dev>
Date: Tue, 23 Jul 2024 19:53:18 +0200
Subject: [PATCH] [test] Make `xxhash` benchmark target name less generic

Downstream users reported that the `xxhash` target name conflicts with
some of their existing build targets. Rename it to `bench-xxhash`.

See https://github.com/llvm/llvm-project/pull/99634#discussion_r1687666875
---
 llvm/benchmarks/CMakeLists.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/benchmarks/CMakeLists.txt b/llvm/benchmarks/CMakeLists.txt
index 52d726451ada9..fb94892852ccc 100644
--- a/llvm/benchmarks/CMakeLists.txt
+++ b/llvm/benchmarks/CMakeLists.txt
@@ -2,4 +2,4 @@ set(LLVM_LINK_COMPONENTS
   Support)
 
 add_benchmark(DummyYAML DummyYAML.cpp PARTIAL_SOURCES_INTENDED)
-add_benchmark(xxhash xxhash.cpp PARTIAL_SOURCES_INTENDED)
+add_benchmark(bench-xxhash xxhash.cpp PARTIAL_SOURCES_INTENDED)



More information about the llvm-commits mailing list