Skip to content
  • Duncan Sands's avatar
    Tweak the cmake interaction between CMAKE_BUILD_TYPE and LLVM_ENABLE_ASSERTIONS. · 80f122fe
    Duncan Sands authored
    The issue is that CMAKE_BUILD_TYPE=RelWithDebInfo LLVM_ENABLE_ASSERTIONS=ON was
    not building with assertions enabled.  (I was unable to find what in the LLVM
    source tree was adding -DNDEBUG to the build line in this case, so decided that
    it must be cmake itself that was adding it - this may depend on the cmake
    version).  The fix treats any mode that is not Debug as being the same as
    Release for this purpose (previously it was being assumed that cmake would only
    add -DNDEBUG for Release and not for RelWithDebInfo or MinSizeRel).  If other
    versions of cmake don't add -DNDEBUG for RelWithDebInfo then that's OK: with
    this change you just get a useless but harmless -UNDEBUG or -DNDEBUG.
    
    llvm-svn: 186499
    80f122fe
Loading