[all-commits] [llvm/llvm-project] b2ba87: [lldb] Split DynamicLoader binary loading into loc...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Thu Aug 6 09:20:45 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b2ba87ae2551808b36fc74dfac8199d54c043b4a
      https://github.com/llvm/llvm-project/commit/b2ba87ae2551808b36fc74dfac8199d54c043b4a
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2026-08-06 (Thu, 06 Aug 2026)

  Changed paths:
    M lldb/include/lldb/Target/DynamicLoader.h
    M lldb/source/Core/DynamicLoader.cpp
    M lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
    M lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
    M lldb/source/Plugins/Process/mach-core/ProcessMachCore.cpp

  Log Message:
  -----------
  [lldb] Split DynamicLoader binary loading into locate and load (NFCI) (#214372)

LoadBinaryWithUUIDAndAddress both searched for a binary and registered
it with the Target. Split it into LocateBinaries, which only searches,
and LoadBinaryInTarget, which mutates the Target, with
LocateAndLoadBinary keeping the single binary case a one-liner.

The eight binary parameters and the results of the search are bundled in
a new BinarySpec struct, and both entry points return an llvm::Expected.

The motivation is a follow-up that runs LocateBinaries in parallel on
the thread pool. NFC, except for some small improvements to the error
handling because we don't write to the async output stream directly (and
fixed the newline).



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list