50949ebf52
Expose u target API mutex through the SB API. This is motivated by lldb-dap, which is built on top of the SB API and needs a way to execute a series of SB API calls in an atomic manner (see #131242). We can solve this problem by either introducing an additional layer of locking at the DAP level or by exposing the existing locking at the SB API level. This patch implements the second approach. This was discussed in an RFC on Discourse [0]. The original implementation exposed a move-only lock rather than a mutex [1] which doesn't work well with SWIG 4.0 [2]. This implement the alternative solution of exposing the mutex rather than the lock. The SBMutex conforms to the BasicLockable requirement [3] (which is why the methods are called `lock` and `unlock` rather than Lock and Unlock) so it can be used as `std::lock_guard<lldb::SBMutex>` and `std::unique_lock<lldb::SBMutex>`. [0]: https://discourse.llvm.org/t/rfc-exposing-the-target-api-lock-through-the-sb-api/85215/6 [1]: https://github.com/llvm/llvm-project/pull/131404 [2]: https://discourse.llvm.org/t/rfc-bumping-the-minimum-swig-version-to-4-1-0/85377/9 [3]: https://en.cppreference.com/w/cpp/named_req/BasicLockable
23 lines
539 B
CMake
23 lines
539 B
CMake
add_lldb_unittest(APITests
|
|
SBCommandInterpreterTest.cpp
|
|
SBLineEntryTest.cpp
|
|
SBMutexTest.cpp
|
|
|
|
LINK_LIBS
|
|
liblldb
|
|
)
|
|
|
|
# Build with -Wdocumentation. This relies on the tests including all the API
|
|
# headers through API/LLDB.h.
|
|
check_cxx_compiler_flag("-Wdocumentation"
|
|
CXX_SUPPORTS_DOCUMENTATION)
|
|
if (CXX_SUPPORTS_DOCUMENTATION)
|
|
target_compile_options(APITests
|
|
PRIVATE -Wdocumentation)
|
|
endif()
|
|
|
|
|
|
if(Python3_RPATH)
|
|
set_property(TARGET APITests APPEND PROPERTY BUILD_RPATH "${Python3_RPATH}")
|
|
endif()
|