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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

This makes sense but I think the main justification is that many projects do this and therefore defaulting to Release more likely meets users' expectation.

E.g. tensorflow lite has something like

  if(NOT CMAKE_BUILD_TYPE)
    message(STATUS "Setting build type to Release, for debug builds use"
      "'-DCMAKE_BUILD_TYPE=Debug'.")
    set(CMAKE_BUILD_TYPE "Release")
  endif()


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124153