From 5342ecf364e633960e84ca021ef44f1274e9938c Mon Sep 17 00:00:00 2001 From: scheibelp Date: Fri, 1 Sep 2017 10:32:04 -0700 Subject: Set default cmake build_type to Release for llvm Override CMake "build_type" variant to default to "Release" for llvm package. --- lib/spack/spack/build_systems/cmake.py | 2 +- var/spack/repos/builtin/packages/llvm/package.py | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/spack/spack/build_systems/cmake.py b/lib/spack/spack/build_systems/cmake.py index f643320b10..9160390a06 100644 --- a/lib/spack/spack/build_systems/cmake.py +++ b/lib/spack/spack/build_systems/cmake.py @@ -84,7 +84,7 @@ class CMakePackage(PackageBase): # https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html variant('build_type', default='RelWithDebInfo', - description='The build type to build', + description='CMake build type', values=('Debug', 'Release', 'RelWithDebInfo', 'MinSizeRel')) depends_on('cmake', type='build') diff --git a/var/spack/repos/builtin/packages/llvm/package.py b/var/spack/repos/builtin/packages/llvm/package.py index 64e50f0cfb..396cdbcb0c 100644 --- a/var/spack/repos/builtin/packages/llvm/package.py +++ b/var/spack/repos/builtin/packages/llvm/package.py @@ -70,6 +70,9 @@ class Llvm(CMakePackage): variant('all_targets', default=True, description="Build all supported targets, default targets " ",NVPTX,AMDGPU,CppBackend") + variant('build_type', default='Release', + description='CMake build type', + values=('Debug', 'Release', 'RelWithDebInfo', 'MinSizeRel')) # Build dependency depends_on('cmake@3.4.3:', type='build') -- cgit v1.2.3-70-g09d2