From c0ea5d8ce50100b7e26a316a0fce887baf64a835 Mon Sep 17 00:00:00 2001 From: nanaya Date: Wed, 5 Jul 2017 19:43:47 +0900 Subject: [PATCH] tools: always include llvm_version in config Also used in common.gypi to check whether a flag is needed or not based on llvm version. PR-URL: https://github.com/nodejs/node/pull/14077 Reviewed-By: Ben Noordhuis Reviewed-By: Refael Ackermann --- configure | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 9a0855b49ad2e0..a481e444330c06 100755 --- a/configure +++ b/configure @@ -640,12 +640,14 @@ def check_compiler(o): # to a version that is not completely ancient. warn('C compiler too old, need gcc 4.2 or clang 3.2 (CC=%s)' % CC) - # Need llvm_version or gas_version when openssl asm files are compiled + if is_clang: + o['variables']['llvm_version'] = get_llvm_version(CC) + + # Need xcode_version or gas_version when openssl asm files are compiled. if options.without_ssl or options.openssl_no_asm or options.shared_openssl: return if is_clang: - o['variables']['llvm_version'] = get_llvm_version(CC) if sys.platform == 'darwin': o['variables']['xcode_version'] = get_xcode_version(CC) else: