zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Set default cmake build_type to Release for llvm

Browse Source 
Override CMake "build_type" variant to default to "Release" for
llvm package.
This commit is contained in:
scheibelp 2017-09-01 10:32:04 -07:00 committed by GitHub
parent 48599837d8
commit 5342ecf364
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/spack/spack/build_systems/cmake.py
View File

@ -84,7 +84,7 @@ class CMakePackage(PackageBase):
# https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html # https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html
variant('build_type', default='RelWithDebInfo', variant('build_type', default='RelWithDebInfo',
description='The build type to build', description='CMake build type',
values=('Debug', 'Release', 'RelWithDebInfo', 'MinSizeRel')) values=('Debug', 'Release', 'RelWithDebInfo', 'MinSizeRel'))
depends_on('cmake', type='build') depends_on('cmake', type='build')

3
var/spack/repos/builtin/packages/llvm/package.py
View File

@ -70,6 +70,9 @@ class Llvm(CMakePackage):
variant('all_targets', default=True, variant('all_targets', default=True,
description="Build all supported targets, default targets " description="Build all supported targets, default targets "
"<current arch>,NVPTX,AMDGPU,CppBackend") "<current arch>,NVPTX,AMDGPU,CppBackend")
variant('build_type', default='Release',
description='CMake build type',
values=('Debug', 'Release', 'RelWithDebInfo', 'MinSizeRel'))
# Build dependency # Build dependency
depends_on('cmake@3.4.3:', type='build') depends_on('cmake@3.4.3:', type='build')