[all-commits] [llvm/llvm-project] c074f5: [DWARFLinker][NFC] Remove usages of "const object:...

avl-llvm via All-commits all-commits at lists.llvm.org
Fri Feb 28 02:26:28 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c074f5234d29439116f0e0be6033ea9331e85394
      https://github.com/llvm/llvm-project/commit/c074f5234d29439116f0e0be6033ea9331e85394
  Author: Alexey Lapshin <a.v.lapshin at mail.ru>
  Date:   2020-02-28 (Fri, 28 Feb 2020)

  Changed paths:
    M llvm/include/llvm/DWARFLinker/DWARFLinker.h
    M llvm/lib/DWARFLinker/DWARFLinker.cpp
    M llvm/tools/dsymutil/DwarfLinkerForBinary.cpp
    M llvm/tools/dsymutil/DwarfLinkerForBinary.h
    M llvm/tools/dsymutil/DwarfStreamer.cpp
    M llvm/tools/dsymutil/DwarfStreamer.h

  Log Message:
  -----------
  [DWARFLinker][NFC] Remove usages of "const object::ObjectFile" from DWARFLinker.

Summary:
DWARFContext has all the required information to access source debug info.
It is not necessary to use "const object::ObjectFile" to create DWARFContext.
Thus this patch removes all usages of "const object::ObjectFile"
from DWARFLinker. Instead, already created DWARFContext is passed
to DWARFLinker. The purpose is to not depend on "const object::ObjectFile".

The patch looks big, but most of changes are renamings and movements.

Testing: it passes "check-all" lit testing. MD5 checksum for clang .dSYM bundle
matches for the dsymutil with/without that patch.

Reviewers: JDevlieghere, friss, dblaikie, aprantl

Reviewed By: JDevlieghere

Subscribers: hiraditya, llvm-commits

Tags: #llvm, #debug-info

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




More information about the All-commits mailing list