Commit 0e02977b authored by Sourabh Singh Tomar's avatar Sourabh Singh Tomar
Browse files

Recommit "[DWARF] Support for loclist.dwo section in llvm and llvm-dwarfdump." 

The original commit message follows.

This patch adds support for debug_loclists.dwo section in llvm and llvm-dwarfdump.
Also Fixes PR43622, PR43623.

Reviewers: dblaikie, probinson, labath, aprantl, jini.susan.george

Differential Revision: https://reviews.llvm.org/D69462
parent 02cb4b2f

llvm/include/llvm/DebugInfo/DWARF/DWARFObject.h

+1 −0
Original line number Diff line number Diff line
@@ -60,6 +60,7 @@ public: 
  virtual StringRef getAbbrevDWOSection() const { return ""; }

  virtual const DWARFSection &getLineDWOSection() const { return Dummy; }

  virtual const DWARFSection &getLocDWOSection() const { return Dummy; }

  virtual const DWARFSection &getLoclistsDWOSection() const { return Dummy; }

  virtual StringRef getStrDWOSection() const { return ""; }

  virtual const DWARFSection &getStrOffsetsDWOSection() const {

    return Dummy;

llvm/include/llvm/MC/MCObjectFileInfo.h

+5 −1
Original line number Diff line number Diff line
@@ -120,8 +120,9 @@ protected: 
  /// The DWARF v5 locations list section.

  MCSection *DwarfLoclistsSection = nullptr;



  /// The DWARF v5 range list section for fission.

  /// The DWARF v5 range and location list sections for fission.

  MCSection *DwarfRnglistsDWOSection = nullptr;

  MCSection *DwarfLoclistsDWOSection = nullptr;



  // These are for Fission DWP files.

  MCSection *DwarfCUIndexSection = nullptr;
@@ -299,6 +300,9 @@ public: 
  MCSection *getDwarfRnglistsDWOSection() const {

    return DwarfRnglistsDWOSection;

  }

  MCSection *getDwarfLoclistsDWOSection() const {

    return DwarfLoclistsDWOSection;

  }

  MCSection *getDwarfCUIndexSection() const { return DwarfCUIndexSection; }

  MCSection *getDwarfTUIndexSection() const { return DwarfTUIndexSection; }

  MCSection *getDwarfSwiftASTSection() const { return DwarfSwiftASTSection; }

llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp

+37 −18
Original line number Diff line number Diff line
@@ -1158,8 +1158,9 @@ void DwarfDebug::finalizeModuleInfo() { 
      if (U.hasRangeLists())

        U.addRnglistsBase();



      if (!DebugLocs.getLists().empty() && !useSplitDwarf()) {

      if (!DebugLocs.getLists().empty()) {

        DebugLocs.setSym(Asm->createTempSymbol("loclists_table_base"));

        if (!useSplitDwarf())

          U.addSectionLabel(U.getUnitDie(), dwarf::DW_AT_loclists_base,

                            DebugLocs.getSym(),

                            TLOF.getDwarfLoclistsSection()->getBeginSymbol());
@@ -1207,9 +1208,10 @@ void DwarfDebug::endModule() { 
  emitDebugStr();



  if (useSplitDwarf())

    // Handles debug_loc.dwo / debug_loclists.dwo section emission

    emitDebugLocDWO();

  else

    // Emit info into a debug loc section.

    // Handles debug_loc / debug_loclists section emission

    emitDebugLoc();



  // Corresponding abbreviations into a abbrev section.
@@ -2335,8 +2337,6 @@ static MCSymbol *emitLoclistsTableHeader(AsmPrinter *Asm, 


  const auto &DebugLocs = DD.getDebugLocs();



  // FIXME: Generate the offsets table and use DW_FORM_loclistx with the

  // DW_AT_loclists_base attribute. Until then set the number of offsets to 0.

  Asm->OutStreamer->AddComment("Offset entry count");

  Asm->emitInt32(DebugLocs.getLists().size());

  Asm->OutStreamer->EmitLabel(DebugLocs.getSym());
@@ -2443,27 +2443,29 @@ static void emitRangeList( 
  }

}



// Handles emission of both debug_loclist / debug_loclist.dwo

static void emitLocList(DwarfDebug &DD, AsmPrinter *Asm, const DebugLocStream::List &List) {

  emitRangeList(

      DD, Asm, List.Label, DD.getDebugLocs().getEntries(List), *List.CU,

      dwarf::DW_LLE_base_addressx, dwarf::DW_LLE_offset_pair,

      dwarf::DW_LLE_startx_length, dwarf::DW_LLE_end_of_list,

      llvm::dwarf::LocListEncodingString,

  emitRangeList(DD, Asm, List.Label, DD.getDebugLocs().getEntries(List),

                *List.CU, dwarf::DW_LLE_base_addressx,

                dwarf::DW_LLE_offset_pair, dwarf::DW_LLE_startx_length,

                dwarf::DW_LLE_end_of_list, llvm::dwarf::LocListEncodingString,

                /* ShouldUseBaseAddress */ true,

                [&](const DebugLocStream::Entry &E) {

                  DD.emitDebugLocEntryLocation(E, List.CU);

                });

}



// Emit locations into the .debug_loc/.debug_rnglists section.

// Emit locations into the .debug_loc/.debug_loclists section.

void DwarfDebug::emitDebugLoc() {

  if (DebugLocs.getLists().empty())

    return;



  MCSymbol *TableEnd = nullptr;

  if (getDwarfVersion() >= 5) {



    Asm->OutStreamer->SwitchSection(

        Asm->getObjFileLowering().getDwarfLoclistsSection());



    TableEnd = emitLoclistsTableHeader(Asm, *this);

  } else {

    Asm->OutStreamer->SwitchSection(
@@ -2477,11 +2479,29 @@ void DwarfDebug::emitDebugLoc() { 
    Asm->OutStreamer->EmitLabel(TableEnd);

}



// Emit locations into the .debug_loc.dwo/.debug_loclists.dwo section.

void DwarfDebug::emitDebugLocDWO() {

  if (DebugLocs.getLists().empty())

    return;



  if (getDwarfVersion() >= 5) {

    MCSymbol *TableEnd = nullptr;

    Asm->OutStreamer->SwitchSection(

        Asm->getObjFileLowering().getDwarfLoclistsDWOSection());

    TableEnd = emitLoclistsTableHeader(Asm, *this);

    for (const auto &List : DebugLocs.getLists())

      emitLocList(*this, Asm, List);



    if (TableEnd)

      Asm->OutStreamer->EmitLabel(TableEnd);

    return;

  }



  for (const auto &List : DebugLocs.getLists()) {

    Asm->OutStreamer->SwitchSection(

        Asm->getObjFileLowering().getDwarfLocDWOSection());

    Asm->OutStreamer->EmitLabel(List.Label);



    for (const auto &Entry : DebugLocs.getEntries(List)) {

      // GDB only supports startx_length in pre-standard split-DWARF.

      // (in v5 standard loclists, it currently* /only/ supports base_address +
@@ -2494,7 +2514,6 @@ void DwarfDebug::emitDebugLocDWO() { 
      unsigned idx = AddrPool.getIndex(Entry.Begin);

      Asm->EmitULEB128(idx);

      Asm->EmitLabelDifference(Entry.End, Entry.Begin, 4);



      emitDebugLocEntryLocation(Entry, List.CU);

    }

    Asm->emitInt8(dwarf::DW_LLE_end_of_list);

llvm/lib/DebugInfo/DWARF/DWARFContext.cpp

+13 −0
Original line number Diff line number Diff line
@@ -403,6 +403,14 @@ void DWARFContext::dump( 
                            0);

    dumpLoclistsSection(OS, LLDumpOpts, Data, getRegisterInfo(), *Off);

  }

  if (const auto *Off =

          shouldDump(ExplicitDWO, ".debug_loclists.dwo", DIDT_ID_DebugLoclists,

                     DObj->getLoclistsDWOSection().Data)) {

    DWARFDataExtractor Data(*DObj, DObj->getLoclistsDWOSection(),

                            isLittleEndian(), 0);

    dumpLoclistsSection(OS, LLDumpOpts, Data, getRegisterInfo(), *Off);

  }



  if (const auto *Off =

          shouldDump(ExplicitDWO, ".debug_loc.dwo", DIDT_ID_DebugLoc,

                     DObj->getLocDWOSection().Data)) {
@@ -1385,6 +1393,7 @@ class DWARFObjInMemory final : public DWARFObject { 


  DWARFSectionMap LocSection;

  DWARFSectionMap LoclistsSection;

  DWARFSectionMap LoclistsDWOSection;

  DWARFSectionMap LineSection;

  DWARFSectionMap RangesSection;

  DWARFSectionMap RnglistsSection;
@@ -1411,6 +1420,7 @@ class DWARFObjInMemory final : public DWARFObject { 
    return StringSwitch<DWARFSectionMap *>(Name)

        .Case("debug_loc", &LocSection)

        .Case("debug_loclists", &LoclistsSection)

        .Case("debug_loclists.dwo", &LoclistsDWOSection)

        .Case("debug_line", &LineSection)

        .Case("debug_frame", &FrameSection)

        .Case("eh_frame", &EHFrameSection)
@@ -1741,6 +1751,9 @@ public: 
  const DWARFSection &getRnglistsDWOSection() const override {

    return RnglistsDWOSection;

  }

  const DWARFSection &getLoclistsDWOSection() const override {

    return LoclistsDWOSection;

  }

  const DWARFSection &getAddrSection() const override { return AddrSection; }

  StringRef getCUIndexSection() const override { return CUIndexSection; }

  StringRef getGdbIndexSection() const override { return GdbIndexSection; }

llvm/lib/DebugInfo/DWARF/DWARFDie.cpp

+1 −0
Original line number Diff line number Diff line
@@ -89,6 +89,7 @@ static void dumpLocation(raw_ostream &OS, DWARFFormValue &FormValue, 


    if (FormValue.getForm() == DW_FORM_loclistx) {

      FormValue.dump(OS, DumpOpts);



      if (auto LoclistOffset = U->getLoclistOffset(Offset))

        Offset = *LoclistOffset + U->getLocSectionBase();

      else