[all-commits] [llvm/llvm-project] b4d710: [lldb][Android] Use a lambda for calls to ::open i...

[Kazuki Sakamoto via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b4d710e410595905c6c1d40cd5d257dfa9143bbe
      https://github.com/llvm/llvm-project/commit/b4d710e410595905c6c1d40cd5d257dfa9143bbe
  Author: Kazuki Sakamoto <sakamoto at splhack.org>
  Date:   2023-06-15 (Thu, 15 Jun 2023)

  Changed paths:
    M lldb/include/lldb/Host/FileSystem.h
    M lldb/source/Host/common/PseudoTerminal.cpp
    M lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp
    M lldb/source/Host/posix/FileSystemPosix.cpp
    M lldb/source/Host/posix/PipePosix.cpp
    M lldb/source/Host/posix/ProcessLauncherPosixFork.cpp

  Log Message:
  -----------
  [lldb][Android] Use a lambda for calls to ::open in RetryAfterSignal

lldb-server for Android does not build with NDK r21 and above due to
RetryAfterSignal and Bionic ::open mismatch.
https://github.com/llvm/llvm-project/issues/54727

Apply the LLVM patch to LLDB.
https://github.com/llvm/llvm-project/commit/0a0e411204a2baa520fd73a8d69b664f98b428ba

> In Bionic, open can be overloaded for _FORTIFY_SOURCE support, causing
> compile errors of RetryAfterSignal due to overload resolution. Wrapping
> the call in a lambda avoids this.

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