Skip to content

Commit

Permalink
Merge pull request #227 from rgkirch/patch-1
Browse files Browse the repository at this point in the history
remove platform dependent use of 'sed' for getting major and minor llvm versions. using cmake regex instead
  • Loading branch information
tomhrr authored Apr 16, 2024
2 parents 46e6dfc + a6093b2 commit c7dce17
Showing 1 changed file with 4 additions and 10 deletions.
14 changes: 4 additions & 10 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,16 +27,10 @@ execute_process (COMMAND ${LLVM_CONFIG} --version
execute_process (COMMAND ${LLVM_CONFIG} --bindir
OUTPUT_VARIABLE LLVM_BIN_DIR
OUTPUT_STRIP_TRAILING_WHITESPACE)
execute_process (COMMAND echo ${D_LLVM_VERSION}
COMMAND sed -e "s/[^0-9].*//"
OUTPUT_VARIABLE D_LLVM_VERSION_MAJOR
OUTPUT_STRIP_TRAILING_WHITESPACE)
execute_process (COMMAND echo ${D_LLVM_VERSION}
COMMAND sed -e "s/[0-9]*\\.//"
COMMAND sed -e "s/svn//"
COMMAND sed -e "s/\\..*//"
OUTPUT_VARIABLE D_LLVM_VERSION_MINOR
OUTPUT_STRIP_TRAILING_WHITESPACE)

string(REGEX MATCH "[^0-9]*([^.]+)\.([^.]+).*" _ ${D_LLVM_VERSION})
set(D_LLVM_VERSION_MAJOR ${CMAKE_MATCH_1})
set(D_LLVM_VERSION_MINOR ${CMAKE_MATCH_2})

if (NOT D_LLVM_VERSION)
message (FATAL_ERROR "Unable to retrieve version number from llvm-config executable (${LLVM_CONFIG})")
Expand Down

0 comments on commit c7dce17

Please sign in to comment.