[all-commits] [llvm/llvm-project] a895a4: [lldb/test] Simplify/generalize YAMLModuleTester

[Pavel Labath via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a895a446bcde4d78572ced8ff0b559f476aa4274
      https://github.com/llvm/llvm-project/commit/a895a446bcde4d78572ced8ff0b559f476aa4274
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2020-10-30 (Fri, 30 Oct 2020)

  Changed paths:
    M lldb/unittests/Expression/DWARFExpressionTest.cpp
    M lldb/unittests/SymbolFile/DWARF/DWARFASTParserClangTests.cpp
    M lldb/unittests/TestingSupport/Symbol/CMakeLists.txt
    M lldb/unittests/TestingSupport/Symbol/YAMLModuleTester.cpp
    M lldb/unittests/TestingSupport/Symbol/YAMLModuleTester.h

  Log Message:
  -----------
  [lldb/test] Simplify/generalize YAMLModuleTester

The class only supports a single DWARF unit (needed for my new test), and it
reimplements chunks of object and symbol file classes. We can just make it use
the real thing, save some LOC and get the full feature set.

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


  Commit: 62286c569d84c81236522c386a67e60f8f333f63
      https://github.com/llvm/llvm-project/commit/62286c569d84c81236522c386a67e60f8f333f63
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2020-10-30 (Fri, 30 Oct 2020)

  Changed paths:
    M lldb/test/API/tools/lldb-server/register-reading/TestGdbRemoteGPacket.py

  Log Message:
  -----------
  [lldb/test] Remove a double debugserver launch in TestGdbRemoteGPacket

Debug server is already launched by prep_debug_monitor_and_inferior. The
second seems to have been benign so far, but after 8cc49bec2 this test
started failing frequently on GreenDragon, and this is the only unusual
thing about it.


Compare: https://github.com/llvm/llvm-project/compare/d085697013b4...62286c569d84