[all-commits] [llvm/llvm-project] 75e93e: [lldb][TerminalTest] Fix assertion failure

Kazuki Sakamoto via All-commits all-commits at lists.llvm.org
Thu Jun 15 21:33:08 PDT 2023


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

  Changed paths:
    M lldb/unittests/Host/posix/TerminalTest.cpp

  Log Message:
  -----------
  [lldb][TerminalTest] Fix assertion failure

D152712 replaced `llvm::sys::RetryAfterSignal(-1, ::open)` with
`FileSystem::Instance().Open` for bionic in PseudoTerminal::OpenSecondary, and
FileSystem::Instance() is failing with assertion on arm Linux.

The assertion should be FileSystem re-initialization check, therefore the
hypothesis is that TerminalTest tests are initializing FileSystem instance
repeatedly.

Use SubsystemRAII<FileSystem> to ensure tearing down the FileSystem instance.

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




More information about the All-commits mailing list