[PATCH] D124153: [CMake] Change default CMAKE_BULID_TYPE to Release

[Tobias Hieta via Phabricator via llvm-commits]

thieta created this revision.
thieta added reviewers: hans, tstellar, nikic, mstorsjo, mehdi_amini, zero9178, stephenneuendorffer, tschuett, hubert.reinterpretcast.
Herald added a subscriber: mgorny.
Herald added a project: All.
thieta requested review of this revision.
Herald added a project: LLVM.

This is a better default for users of LLVM, since Debug builds
are slow, big and prone to OOM the linker.

Discussion on discourse here:
https://discourse.llvm.org/t/rfc-select-a-better-linker-by-default-or-warn-about-using-bfd


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D124153

Files:
  llvm/CMakeLists.txt


Index: llvm/CMakeLists.txt
===================================================================
--- llvm/CMakeLists.txt
+++ llvm/CMakeLists.txt
@@ -64,8 +64,8 @@
 endif()
 
 if (NOT CMAKE_BUILD_TYPE AND NOT CMAKE_CONFIGURATION_TYPES)
-  message(STATUS "No build type selected, default to Debug")
-  set(CMAKE_BUILD_TYPE "Debug" CACHE STRING "Build type (default Debug)" FORCE)
+  message(STATUS "No build type selected, default to Release")
+  set(CMAKE_BUILD_TYPE "Release" CACHE STRING "Build type (default Release)" FORCE)
 endif()
 
 # Side-by-side subprojects layout: automatically set the


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124153.424116.patch
Type: text/x-patch
Size: 594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220421/c45fad85/attachment.bin>