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

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

In D124153#3468300 <https://reviews.llvm.org/D124153#3468300>, @mehdi_amini wrote:

> We have some pieces of data though: the number of people who complain about the build filling up their disk and the link taking forever (or OOM). On the other hand the people who "know" are already setting up an explicit value...
> That said I tend to think that the audience is heterogeneous enough that I agree it seems hard to make sure we "pick a default that's "right" " in the absolute (hence my proposal to not pick any).

I think we can pick a 'right' default, we just need to define 'right' differently to make this possible. :)  I think the 'right' default is the one that is easiest to build, which is the release build.  It takes less resources (CPU, memory, and disk space) and takes less time to build.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124153