diff options
author | Massimiliano Culpo <massimiliano.culpo@gmail.com> | 2023-05-04 17:33:35 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-04 11:33:35 -0400 |
commit | 86d3bad1e05140398f6e19919de38928f3d0c484 (patch) | |
tree | f1bc0fa88f22ce038298e57bb6868be5e25110bf /.readthedocs.yml | |
parent | 600955edd4ca22b2d94e725706fc4733d5c4e4d9 (diff) | |
download | spack-86d3bad1e05140398f6e19919de38928f3d0c484.tar.gz spack-86d3bad1e05140398f6e19919de38928f3d0c484.tar.bz2 spack-86d3bad1e05140398f6e19919de38928f3d0c484.tar.xz spack-86d3bad1e05140398f6e19919de38928f3d0c484.zip |
cmake build system: change default build type to `Release` (#36679)
This switches the default Make build type to `build_type=Release`.
This offers:
- higher optimization level, including loop vectorization on older GCC
- adds NDEBUG define, which disables assertions, which could cause speedups if assertions are in loops etc
- no `-g` means smaller install size
Downsides are:
- worse backtraces (though this does NOT strip symbols)
- perf reports may be useless
- no function arguments / local variables in debugger (could be of course)
- no file path / line numbers in debugger
The downsides can be mitigated by overriding to `build_type=RelWithDebInfo` in `packages.yaml`,
if needed. The upside is that builds will be MUCH smaller (and faster) with this change.
---------
Co-authored-by: Gregory Becker <becker33@llnl.gov>
Diffstat (limited to '.readthedocs.yml')
0 files changed, 0 insertions, 0 deletions