[all-commits] [llvm/llvm-project] 39385d: [CodeGen][Debuginfo][NFC] Refactor DIE values Size...

avl-llvm via All-commits all-commits at lists.llvm.org
Wed Jan 12 02:16:41 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 39385d4cd1c6e70f9b7c9cadb35e57efbdf2ecbc
      https://github.com/llvm/llvm-project/commit/39385d4cd1c6e70f9b7c9cadb35e57efbdf2ecbc
  Author: Alexey Lapshin <a.v.lapshin at mail.ru>
  Date:   2022-01-12 (Wed, 12 Jan 2022)

  Changed paths:
    M llvm/include/llvm/BinaryFormat/Dwarf.h
    M llvm/include/llvm/CodeGen/AsmPrinter.h
    M llvm/include/llvm/CodeGen/DIE.h
    M llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    M llvm/lib/CodeGen/AsmPrinter/DIE.cpp
    M llvm/lib/CodeGen/AsmPrinter/DIEHash.cpp
    M llvm/lib/CodeGen/AsmPrinter/DwarfFile.cpp
    M llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
    M llvm/unittests/CodeGen/DIETest.cpp
    M llvm/unittests/DebugInfo/DWARF/DwarfGenerator.cpp

  Log Message:
  -----------
  [CodeGen][Debuginfo][NFC] Refactor DIE values SizeOf method to not depend on AsmPrinter.

SizeOf() method of DIE values(unsigned SizeOf(const AsmPrinter *AP, dwarf::Form Form) const)
depends on AsmPrinter. AsmPrinter is too specific class here. This patch removes dependency
on AsmPrinter and use dwarf::FormParams structure instead. It allows calculate DIE values
size without using AsmPrinter. That refactoring is useful for D96035([dsymutil][DWARFlinker]
implement separate multi-thread processing for compile units.)

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




More information about the All-commits mailing list