summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorscheibelp <scheibel1@llnl.gov>2017-09-01 10:32:04 -0700
committerGitHub <noreply@github.com>2017-09-01 10:32:04 -0700
commit5342ecf364e633960e84ca021ef44f1274e9938c (patch)
tree06bb335569ae1a9b9b1e260403c79a0ad6b551af
parent48599837d8782dc62aee4e5d20006e5f606103a3 (diff)
downloadspack-5342ecf364e633960e84ca021ef44f1274e9938c.tar.gz
spack-5342ecf364e633960e84ca021ef44f1274e9938c.tar.bz2
spack-5342ecf364e633960e84ca021ef44f1274e9938c.tar.xz
spack-5342ecf364e633960e84ca021ef44f1274e9938c.zip
Set default cmake build_type to Release for llvm
Override CMake "build_type" variant to default to "Release" for llvm package.
-rw-r--r--lib/spack/spack/build_systems/cmake.py2
-rw-r--r--var/spack/repos/builtin/packages/llvm/package.py3
2 files changed, 4 insertions, 1 deletions
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 "
"<current arch>,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')