[llvm] [BOLT] Fix LLVM_APPEND_VC_REV support (PR #142410)
Paschalis Mpeis via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 2 09:32:01 PDT 2025
https://github.com/paschalis-mpeis updated https://github.com/llvm/llvm-project/pull/142410
>From cf25d24a0170e99d1fdd635ab0c2838d584759cc Mon Sep 17 00:00:00 2001
From: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
Date: Mon, 2 Jun 2025 08:37:39 +0100
Subject: [PATCH 1/2] [BOLT] Fix LLVM_APPEND_VC_REV support
The CMake flag LLVM_APPEND_VC_REV can be passed when building BOLT
a BOLT to prevent including a VC Revision. This patch enables this
functionality.
Usage: `-DLLVM_APPEND_VC_REV=OFF` when running CMake.
---
bolt/lib/Utils/CMakeLists.txt | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/bolt/lib/Utils/CMakeLists.txt b/bolt/lib/Utils/CMakeLists.txt
index efba6d54449d3..812c199846080 100644
--- a/bolt/lib/Utils/CMakeLists.txt
+++ b/bolt/lib/Utils/CMakeLists.txt
@@ -6,12 +6,25 @@ set(version_inc "${CMAKE_CURRENT_BINARY_DIR}/VCSVersion.inc")
set(generate_vcs_version_script "${LLVM_CMAKE_DIR}/GenerateVersionFromVCS.cmake")
+if(llvm_vc AND LLVM_APPEND_VC_REV)
+ set(llvm_source_dir ${LLVM_MAIN_SRC_DIR})
+endif()
+if (LLVM_VC_REPOSITORY AND LLVM_VC_REVISION)
+ set(llvm_source_dir ${LLVM_SOURCE_DIR})
+ set(llvm_vc_repository ${LLVM_VC_REPOSITORY})
+ set(llvm_vc_revision ${LLVM_VC_REVISION})
+endif()
+if(bolt_vc AND LLVM_APPEND_VC_REV)
+ set(bolt_source_dir ${BOLT_SOURCE_DIR})
+endif()
+
# Create custom target to generate the VC revision include.
add_custom_command(OUTPUT "${version_inc}"
DEPENDS "${llvm_vc}" "${bolt_vc}" "${generate_vcs_version_script}"
COMMAND ${CMAKE_COMMAND} "-DNAMES=BOLT"
+ "-DLLVM_SOURCE_DIR=${llvm_source_dir}"
+ "-DBOLT_SOURCE_DIR=${bolt_source_dir}"
"-DHEADER_FILE=${version_inc}"
- "-DBOLT_SOURCE_DIR=${BOLT_SOURCE_DIR}"
"-DLLVM_VC_REPOSITORY=${llvm_vc_repository}"
"-DLLVM_VC_REVISION=${llvm_vc_revision}"
"-DLLVM_FORCE_VC_REVISION=${LLVM_FORCE_VC_REVISION}"
>From 8c0ef06a044b9aa9e7c8c79d9331dc2a0a4fe37a Mon Sep 17 00:00:00 2001
From: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
Date: Mon, 2 Jun 2025 17:31:47 +0100
Subject: [PATCH 2/2] consistent spacing
---
bolt/lib/Utils/CMakeLists.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bolt/lib/Utils/CMakeLists.txt b/bolt/lib/Utils/CMakeLists.txt
index 812c199846080..94933644ef5ef 100644
--- a/bolt/lib/Utils/CMakeLists.txt
+++ b/bolt/lib/Utils/CMakeLists.txt
@@ -9,7 +9,7 @@ set(generate_vcs_version_script "${LLVM_CMAKE_DIR}/GenerateVersionFromVCS.cmake"
if(llvm_vc AND LLVM_APPEND_VC_REV)
set(llvm_source_dir ${LLVM_MAIN_SRC_DIR})
endif()
-if (LLVM_VC_REPOSITORY AND LLVM_VC_REVISION)
+if(LLVM_VC_REPOSITORY AND LLVM_VC_REVISION)
set(llvm_source_dir ${LLVM_SOURCE_DIR})
set(llvm_vc_repository ${LLVM_VC_REPOSITORY})
set(llvm_vc_revision ${LLVM_VC_REVISION})
More information about the llvm-commits
mailing list