[all-commits] [llvm/llvm-project] d01fd2: [lldb] Add nodebug attribute to import-std-module/...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Wed Oct 23 09:30:22 PDT 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: d01fd2f35a02cb53a5d9d1a5342b5085c5dce66c
      https://github.com/llvm/llvm-project/commit/d01fd2f35a02cb53a5d9d1a5342b5085c5dce66c
  Author: Raphael Isemann <risemann at apple.com>
  Date:   2019-10-23 (Wed, 23 Oct 2019)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/commands/expression/import-std-module/sysroot/root/usr/include/c++/v1/algorithm

  Log Message:
  -----------
  [lldb] Add nodebug attribute to import-std-module/sysroot test

Summary:
So far we rely on the default argument and the fact that we don't call this
inline function in our actual `main.cpp` to make sure that this function can only
be called if LLDB loads this header as a C++ module. This patch just adds
the nodebug attribute as yet another measure to make sure LLDB can't call this
function without the standard module loaded. Note that the test is already
requiring clang for the sysroot setup, so its fine that this is a Clang specific attribute.

Reviewers: friss, labath

Subscribers: JDevlieghere, lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D68861




More information about the All-commits mailing list