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

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

The target audience for the default configuration should be people who are new to the project, it could be users or it could be developers.  It also could be people with very little experience programming or building from code from source.  We want to keep the barrier for entry as low as possible for this group of people.  We can have the best documentation or have verbose error messages to help guide them in the right direction, but the best way to communicate to new users about how to build the project is to have a sensible default configuration.

I also don't think we should worry about people who complain about the default build configuration.  If someone has enough understanding of the project to know that the default configuration is not the best for them, then they have enough understanding to change the configuration.  The defaults should be designed for people who may not (yet) know anything about build types, debug info, assertions, and who just want to get started using and participating in the project.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124153/new/

https://reviews.llvm.org/D124153