[all-commits] [llvm/llvm-project] 10a507: [lldb] Define lldbassert based on NDEBUG instead o...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Tue May 16 09:27:23 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 10a50762caa6ac17dd063b28863a2ec60576c6f7
      https://github.com/llvm/llvm-project/commit/10a50762caa6ac17dd063b28863a2ec60576c6f7
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2023-05-16 (Tue, 16 May 2023)

  Changed paths:
    M lldb/docs/resources/contributing.rst
    M lldb/include/lldb/Utility/LLDBAssert.h
    M lldb/source/Utility/LLDBAssert.cpp

  Log Message:
  -----------
  [lldb] Define lldbassert based on NDEBUG instead of LLDB_CONFIGURATION_DEBUG

Whether assertions are enabled or not is orthogonal to the build type
which could lead to surprising behavior for lldbassert. Previously, when
doing a debug build with assertions disabled, lldbassert would become a
NOOP, rather than printing an error like it does in a release build. By
definining lldbassert in terms of NDEBUG, it behaves like a regular
assert when assertions are enabled, and like a soft assert.

Differential revision: https://reviews.llvm.org/D150639




More information about the All-commits mailing list