[llvm] r313207 - llvm-dwarfdump: automatically dump both regular and .dwo variant of sections

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 10:44:08 PDT 2017


Also:
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7880/steps/check-llvm%20asan/logs/stdio

==12853==ERROR: AddressSanitizer: heap-buffer-overflow on address
0x6030000040d8 at pc 0x000002e24a57 bp 0x7ffdd544e830 sp
0x7ffdd544e828
READ of size 8 at 0x6030000040d8 thread T0
    #0 0x2e24a56 in
llvm::DWARFVerifier::DieRangeInfo::intersects(llvm::DWARFVerifier::DieRangeInfo
const&) const /mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp:92:27
    #1 0x8cb6d8 in (anonymous namespace)::(anonymous
namespace)::AssertRangesDontIntersect(llvm::DWARFVerifier::DieRangeInfo
const&, std::__1::vector<llvm::DWARFAddressRange,
std::__1::allocator<llvm::DWARFAddressRange> > const&)
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp:2894:3
    #2 0x8c9349 in (anonymous
namespace)::DWARFDebugInfo_TestDWARFDieRangeInfoIntersects_Test::TestBody()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp:2963:3
    #3 0x3281c3e in HandleExceptionsInMethodIfSupported<testing::Test,
void> /mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:2458:12
    #4 0x3281c3e in testing::Test::Run()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:2474
    #5 0x3283f38 in testing::TestInfo::Run()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:2656:11
    #6 0x3285276 in testing::TestCase::Run()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:2774:28
    #7 0x32a5c76 in testing::internal::UnitTestImpl::RunAllTests()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:4649:43
    #8 0x32a4f6e in
HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
bool> /mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:2458:12
    #9 0x32a4f6e in testing::UnitTest::Run()
/mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/utils/unittest/googletest/src/gtest.cc:4257



On Thu, Sep 14, 2017 at 9:15 AM, Yung, Douglas via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Hi Adrian,
>
> That change appears to not have fixed it as it is still failing on the
> build bot as of r313267 on the PS4 Windows bot (http://lab.llvm.org:8011/
> builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/
> builds/12119/steps/test/logs/stdio) and as of r313268 on the PS4 Linux
> bot (http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_
> 64-scei-ps4-ubuntu-fast/builds/17299/steps/test/logs/stdio).
>
> Douglas Yung
>
> > -----Original Message-----
> > From: aprantl at apple.com [mailto:aprantl at apple.com]
> > Sent: Thursday, September 14, 2017 8:45
> > To: Yung, Douglas
> > Cc: llvm-commits at lists.llvm.org
> > Subject: Re: [llvm] r313207 - llvm-dwarfdump: automatically dump both
> regular
> > and .dwo variant of sections
> >
> > Should be fixed by
> >
> > Author: adrian
> > Date: Wed Sep 13 15:30:43 2017
> > New Revision: 313209
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=313209&view=rev
> > Log:
> > Update testcase that was XFAILed on Darwin for llvm-dwarfdump change.
> >
> > Modified:
> >    llvm/trunk/test/DebugInfo/Inputs/gmlt.ll
> >
> > Modified: llvm/trunk/test/DebugInfo/Inputs/gmlt.ll
> > URL: http://llvm.org/viewvc/llvm-
> > project/llvm/trunk/test/DebugInfo/Inputs/gmlt.ll?rev=
> 313209&r1=313208&r2=31320
> > 9&view=diff
> > ============================================================
> ==================
> > --- llvm/trunk/test/DebugInfo/Inputs/gmlt.ll (original)
> > +++ llvm/trunk/test/DebugInfo/Inputs/gmlt.ll Wed Sep 13 15:30:43 2017
> > @@ -89,7 +89,6 @@
> > ; Check that we don't emit any pubnames or pubtypes under -gmlt ;
> CHECK-NOT:
> > .debug_pubnames contents:
> > ; CHECK-NOT: .debug_pubtypes contents:
> > -; CHECK: .apple{{.*}} contents:
> >
> > ; Function Attrs: nounwind uwtable
> > define void @_Z2f1v() #0 !dbg !4 {
> >
> > -- adrian
> >
> > > On Sep 13, 2017, at 6:11 PM, Yung, Douglas <douglas.yung at sony.com>
> wrote:
> > >
> > > Hi Adrian,
> > >
> > > Your commit seems to be causing the PS4 bots (both Windows and Linux)
> to
> > fail on the test DebugInfo\Generic\empty.ll with output similar to this:
> > >
> > > ******************** TEST 'LLVM :: DebugInfo/Generic/empty.ll' FAILED
> > > ********************
> > > Script:
> > > --
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/llc <
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.src/test/DebugInfo/Generic/empty.ll -filetype=obj |
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/llvm-dwarfdump -v - |
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/FileCheck
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.src/test/DebugInfo/Generic/empty.ll
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/llc -split-dwarf-file=foo.dwo <
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.src/test/DebugInfo/Generic/empty.ll -filetype=obj |
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/llvm-dwarfdump -v - |
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.obj/./bin/FileCheck --check-prefix=FISSION
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.src/test/DebugInfo/Generic/empty.ll
> > > --
> > > Exit Code: 1
> > >
> > > Command Output (stderr):
> > > --
> > > /home/buildslave/buildslave1a/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fa
> > > st/llvm.src/test/DebugInfo/Generic/empty.ll:15:10: error: expected
> string
> > not found in input ; CHECK: .debug_pubnames contents:
> > >         ^
> > > <stdin>:40:2: note: scanning from here
> > > version: 2
> > > ^
> > > <stdin>:61:1: note: possible intended match here .debug_str contents:
> > > ^
> > >
> > > Can you take a look?
> > >
> > >
> > >
> > >> -----Original Message-----
> > >> From: llvm-commits [mailto:llvm-commits-bounces at lists.llvm.org] On
> > >> Behalf Of Adrian Prantl via llvm-commits
> > >> Sent: Wednesday, September 13, 2017 15:09
> > >> To: llvm-commits at lists.llvm.org
> > >> Subject: [llvm] r313207 - llvm-dwarfdump: automatically dump both
> > >> regular and .dwo variant of sections
> > >>
> > >> Author: adrian
> > >> Date: Wed Sep 13 15:09:01 2017
> > >> New Revision: 313207
> > >>
> > >> URL: http://llvm.org/viewvc/llvm-project?rev=313207&view=rev
> > >> Log:
> > >> llvm-dwarfdump: automatically dump both regular and .dwo variant of
> > >> sections
> > >>
> > >> Since users typically don't really care about the .dwo / non.dwo
> > >> distinction, this patch makes it so dwarfdump --debug-<info,...>
> > >> dumps .debug_info and (if
> > >> available) also .debug_info.dwo. This simplifies the command line
> > >> interface (I've removed all dwo-specific dump
> > >> options) and makes the tool friendlier to use.
> > >>
> > >> Differential Revision: https://reviews.llvm.org/D37771
> > >>
> > >> Modified:
> > >>    llvm/trunk/include/llvm/BinaryFormat/Dwarf.def
> > >>    llvm/trunk/include/llvm/DebugInfo/DIContext.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFContext.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h
> > >>    llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h
> > >>    llvm/trunk/lib/DebugInfo/DWARF/DWARFContext.cpp
> > >>    llvm/trunk/test/DebugInfo/Inputs/gmlt.ll
> > >>    llvm/trunk/test/DebugInfo/X86/fission-inline.ll
> > >>    llvm/trunk/test/DebugInfo/X86/fission-no-inlining.ll
> > >>    llvm/trunk/test/DebugInfo/X86/fission-ranges.ll
> > >>    llvm/trunk/test/DebugInfo/X86/generate-odr-hash.ll
> > >>    llvm/trunk/test/DebugInfo/X86/gnu-public-names-gmlt.ll
> > >>    llvm/trunk/test/DebugInfo/X86/split-dwarf-cross-unit-reference.ll
> > >>    llvm/trunk/test/DebugInfo/X86/split-dwarf-multiple-cu-hash.ll
> > >>    llvm/trunk/test/DebugInfo/X86/split-dwarf-omit-empty.ll
> > >>    llvm/trunk/test/DebugInfo/dwarfdump-dump-flags.test
> > >>    llvm/trunk/test/DebugInfo/dwarfdump-dwp.test
> > >>    llvm/trunk/test/MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s
> > >>    llvm/trunk/test/MC/ARM/dwarf-asm-no-code.s
> > >>    llvm/trunk/test/MC/ARM/dwarf-asm-nonstandard-section.s
> > >>    llvm/trunk/test/MC/ARM/dwarf-asm-single-section.s
> > >>    llvm/trunk/test/tools/dsymutil/X86/basic-linking-x86.test
> > >>    llvm/trunk/test/tools/llvm-dwp/X86/empty.test
> > >>    llvm/trunk/test/tools/llvm-dwp/X86/simple.test
> > >>    llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
> > >>
> > >> Modified: llvm/trunk/include/llvm/BinaryFormat/Dwarf.def
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/BinaryFormat/Dwarf.def?rev=313207&r1=
> > >> 313206&r2
> > >> =313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/BinaryFormat/Dwarf.def (original)
> > >> +++ llvm/trunk/include/llvm/BinaryFormat/Dwarf.def Wed Sep 13
> > >> +++ 15:09:01
> > >> +++ 2017
> > >> @@ -823,21 +823,16 @@ HANDLE_DW_UT(0x04, skeleton)
> > >> HANDLE_DW_UT(0x05,
> > >> split_compile)  HANDLE_DW_UT(0x06, split_type)
> > >>
> > >> -// DWARF section types. (enum name, ELF name, cmdline name)
> > >> +// DWARF section types. (enum name, ELF name, ELF DWO name, cmdline
> > >> +name)
> > >> // Note that these IDs don't mean anything.
> > >> // TODO: Add Mach-O and COFF names.
> > >> // Official DWARF sections.
> > >> HANDLE_DWARF_SECTION(DebugAbbrev, ".debug_abbrev", "debug-abbrev") -
> > >> HANDLE_DWARF_SECTION(DebugAbbrevDwo, ".debug_abbrev.dwo",
> > >> "debug-abbrev-dwo") HANDLE_DWARF_SECTION(DebugAranges,
> > >> ".debug_aranges", "debug-aranges") HANDLE_DWARF_SECTION(DebugInfo,
> > >> ".debug_info", "debug-info") - HANDLE_DWARF_SECTION(DebugInfoDwo,
> > >> ".debug_info.dwo", "debug-info-dwo") HANDLE_DWARF_SECTION(DebugTypes,
> > >> ".debug_types", "debug-types") - HANDLE_DWARF_SECTION(DebugTypesDwo,
> > >> ".debug_types.dwo", "debug-types-dwo")
> > >> HANDLE_DWARF_SECTION(DebugLine, ".debug_line", "debug-line") -
> > >> HANDLE_DWARF_SECTION(DebugLineDwo, ".debug_line.dwo",
> > >> "debug-line-dwo") HANDLE_DWARF_SECTION(DebugLoc, ".debug_loc",
> > >> "debug-loc") - HANDLE_DWARF_SECTION(DebugLocDwo, ".debug_loc.dwo",
> > >> "debug-loc-dwo") HANDLE_DWARF_SECTION(DebugFrames, ".debug_frames",
> > >> "debug-frames") HANDLE_DWARF_SECTION(DebugMacro, ".debug_macro",
> > >> "debug-macro") HANDLE_DWARF_SECTION(DebugRanges, ".debug_ranges",
> > >> "debug-ranges") @@ -847,8
> > >> +842,6 @@ HANDLE_DWARF_SECTION(DebugGnuPubnames, "
> > >> HANDLE_DWARF_SECTION(DebugGnuPubtypes, ".debug_gnu_pubtypes",
> > >> "debug-gnu-
> > >> pubtypes")  HANDLE_DWARF_SECTION(DebugStr, ".debug_str", "debug-str")
> > >> HANDLE_DWARF_SECTION(DebugStrOffsets, ".debug_str_offsets",
> > >> "debug-str-
> > >> offsets") -HANDLE_DWARF_SECTION(DebugStrDwo, ".debug_str.dwo",
> > >> "debug-str-
> > >> dwo") -HANDLE_DWARF_SECTION(DebugStrOffsetsDwo,
> > >> ".debug_str_offsets.dwo",
> > >> "debug-str-offsets-dwo")  HANDLE_DWARF_SECTION(DebugCUIndex,
> > >> ".debug_cu_index", "debug-cu-index")
> > >> HANDLE_DWARF_SECTION(DebugTUIndex,
> > >> ".debug_tu_index", "debug-tu-index")  // Vendor extensions.
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DIContext.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DIContext.h?rev=313207&r1=3
> > >> 13206&r2=
> > >> 313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DIContext.h (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DIContext.h Wed Sep 13 15:09:01
> > >> +++ 2017
> > >> @@ -122,14 +122,14 @@ enum DIDumpTypeCounter {
> > >>   DIDT_ID_UUID,
> > >>   DIDT_ID_Count
> > >> };
> > >> -static_assert(DIDT_ID_Count <= 64, "section types overflow
> > >> storage");
> > >> +static_assert(DIDT_ID_Count <= 32, "section types overflow
> > >> +storage");
> > >>
> > >> /// Selects which debug sections get dumped.
> > >> -enum DIDumpType : uint64_t {
> > >> +enum DIDumpType : unsigned {
> > >>   DIDT_Null,
> > >> -  DIDT_All             = ~0ULL,
> > >> +  DIDT_All             = ~0U,
> > >> #define HANDLE_DWARF_SECTION(ENUM_NAME, ELF_NAME, CMDLINE_NAME) \
> > >> -  DIDT_##ENUM_NAME = 1 << (DIDT_ID_##ENUM_NAME - 1),
> > >> +  DIDT_##ENUM_NAME = 1U << (DIDT_ID_##ENUM_NAME - 1),
> > >> #include "llvm/BinaryFormat/Dwarf.def"
> > >> #undef HANDLE_DWARF_SECTION
> > >>   DIDT_UUID = 1 << (DIDT_ID_UUID - 1), @@ -138,7 +138,7 @@ enum
> > >> DIDumpType : uint64_t {  /// Container for dump options that control
> > >> which debug information will be  /// dumped.
> > >> struct DIDumpOptions {
> > >> -    uint64_t DumpType = DIDT_All;
> > >> +    unsigned DumpType = DIDT_All;
> > >>     bool DumpEH = false;
> > >>     bool SummarizeTypes = false;
> > >>     bool Verbose = false;
> > >> @@ -158,7 +158,7 @@ public:
> > >>
> > >>   virtual void dump(raw_ostream &OS, DIDumpOptions DumpOpts) = 0;
> > >>
> > >> -  virtual bool verify(raw_ostream &OS, uint64_t DumpType = DIDT_All,
> > >> +  virtual bool verify(raw_ostream &OS, unsigned DumpType = DIDT_All,
> > >>                       DIDumpOptions DumpOpts = {}) {
> > >>     // No verifier? Just say things went well.
> > >>     return true;
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFContext.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFContext.h?rev=31
> > >> 3207&r1=3
> > >> 13206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFContext.h (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFContext.h Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -123,7 +123,7 @@ public:
> > >>
> > >>   void dump(raw_ostream &OS, DIDumpOptions DumpOpts) override;
> > >>
> > >> -  bool verify(raw_ostream &OS, uint64_t DumpType = DIDT_All,
> > >> +  bool verify(raw_ostream &OS, unsigned DumpType = DIDT_All,
> > >>               DIDumpOptions DumpOpts = {}) override;
> > >>
> > >>   using cu_iterator_range =
> > >> DWARFUnitSection<DWARFCompileUnit>::iterator_range;
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h?re
> > >> v=313207&
> > >> r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h
> > >> (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h Wed
> > >> +++ Sep
> > >> +++ 13 15:09:01 2017
> > >> @@ -67,6 +67,7 @@ public:
> > >>
> > >>   void dump(raw_ostream &OS) const;
> > >>   void extract(DataExtractor Data);
> > >> +  bool empty() const { return begin() == end(); }
> > >>
> > >>   DWARFAbbreviationDeclarationSetMap::const_iterator begin() const {
> > >>     return AbbrDeclSets.begin();
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h?rev
> > >> =313207&r
> > >> 1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h
> > >> (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -36,6 +36,9 @@ public:
> > >>   /// data is assumed to be pointing to the beginning of the section.
> > >>   void parse(DataExtractor Data);
> > >>
> > >> +  /// Return whether the section has any entries.
> > >> +  bool empty() const { return Entries.empty(); }
> > >> +
> > >> private:
> > >>   std::vector<std::unique_ptr<FrameEntry>> Entries;  };
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h?rev=3
> > >> 13207&r1=
> > >> 313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h
> > >> (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -63,7 +63,7 @@ public:
> > >>   /// Parse the debug_loc section accessible via the 'data' parameter
> > >> using the
> > >>   /// address size also given in 'data' to interpret the address
> ranges.
> > >>   void parse(const DWARFDataExtractor &data);
> > >> -
> > >> +
> > >>   Optional<LocationList> parseOneLocationList(DWARFDataExtractor
> Data,
> > >>                                               uint32_t *Offset);  };
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h?rev
> > >> =313207&r
> > >> 1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h
> > >> (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -53,6 +53,9 @@ public:
> > >>
> > >>   /// Parse the debug_macinfo section accessible via the 'data'
> parameter.
> > >>   void parse(DataExtractor data);
> > >> +
> > >> +  /// Return whether the section has any entries.
> > >> +  bool empty() const { return Macros.empty(); }
> > >> };
> > >>
> > >> } // end namespace llvm
> > >>
> > >> Modified: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h?rev=
> > >> 313207&r1
> > >> =313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h
> > >> (original)
> > >> +++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -87,6 +87,7 @@ public:
> > >>
> > >>   bool parse(DataExtractor IndexData);
> > >>   void dump(raw_ostream &OS) const;
> > >> +
> > >>   const Entry *getFromOffset(uint32_t Offset) const;
> > >>   const Entry *getFromHash(uint64_t Offset) const;
> > >>
> > >>
> > >> Modified: llvm/trunk/lib/DebugInfo/DWARF/DWARFContext.cpp
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/lib/DebugInfo/DWARF/DWARFContext.cpp?rev=313207&r1
> > >> =313206&r
> > >> 2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/lib/DebugInfo/DWARF/DWARFContext.cpp (original)
> > >> +++ llvm/trunk/lib/DebugInfo/DWARF/DWARFContext.cpp Wed Sep 13
> > >> +++ 15:09:01
> > >> +++ 2017
> > >> @@ -41,6 +41,7 @@
> > >> #include "llvm/Support/Error.h"
> > >> #include "llvm/Support/Format.h"
> > >> #include "llvm/Support/MemoryBuffer.h"
> > >> +#include "llvm/Support/Path.h"
> > >> #include "llvm/Support/TargetRegistry.h"
> > >> #include "llvm/Support/raw_ostream.h"
> > >> #include <algorithm>
> > >> @@ -224,6 +225,10 @@ void DWARFContext::dump(raw_ostream &OS,
> > >>   uint64_t DumpType = DumpOpts.DumpType;
> > >>   bool DumpEH = DumpOpts.DumpEH;
> > >>
> > >> +  StringRef Extension = sys::path::extension(DObj->getFileName());
> > >> +  bool IsDWO = (Extension == ".dwo") || (Extension == ".dwp");
> > >> +
> > >> +  // Print UUID header.
> > >>   const auto *ObjFile = DObj->getFile();
> > >>   if (!(DumpType & DIDT_UUID) || DumpType == DIDT_All)
> > >>     outs() << ObjFile->getFileName() << ":\tfile format "
> > >> @@ -231,72 +236,82 @@ void DWARFContext::dump(raw_ostream &OS,
> > >>   if (DumpType & DIDT_UUID)
> > >>     dumpUUID(OS, *ObjFile);
> > >>
> > >> -  if (DumpType & DIDT_DebugAbbrev) {
> > >> -    OS << ".debug_abbrev contents:\n";
> > >> +  // Print a header for each explicitly-requested section.
> > >> +  // Otherwise just print one for non-empty sections.
> > >> +  bool Explicit = DumpType != DIDT_All && !IsDWO;  auto shouldDump =
> > >> + [&](unsigned DIDT_Section, StringRef Section) {
> > >> +    return (DumpType & DIDT_Section) && (Explicit ||
> > >> + !Section.empty()); };  // Only print empty .dwo section headers when
> > dumping a .dwo file.
> > >> +  bool ExplicitDWO = Explicit && IsDWO;  auto shouldDumpDWO =
> > >> + [&](unsigned DIDT_Section, StringRef Section) {
> > >> +    return (DumpType & DIDT_Section) && (ExplicitDWO ||
> > >> + !Section.empty());  };
> > >> +
> > >> +  // Dump individual sections.
> > >> +  if (shouldDump(DIDT_DebugAbbrev, DObj->getAbbrevSection())) {
> > >> +    OS << "\n.debug_abbrev contents:\n";
> > >>     getDebugAbbrev()->dump(OS);
> > >>   }
> > >> +  if (shouldDumpDWO(DIDT_DebugAbbrev, DObj->getAbbrevDWOSection()))
> {
> > >> +    OS << "\n.debug_abbrev.dwo contents:\n";
> > >> +    getDebugAbbrevDWO()->dump(OS);
> > >> +  }
> > >>
> > >> -  if (DumpType & DIDT_DebugAbbrevDwo)
> > >> -    if (const DWARFDebugAbbrev *D = getDebugAbbrevDWO()) {
> > >> -      OS << "\n.debug_abbrev.dwo contents:\n";
> > >> -      D->dump(OS);
> > >> -    }
> > >> -
> > >> -  if (DumpType & DIDT_DebugInfo) {
> > >> +  if (shouldDump(DIDT_DebugInfo, DObj->getInfoSection().Data)) {
> > >>     OS << "\n.debug_info contents:\n";
> > >>     for (const auto &CU : compile_units())
> > >>       CU->dump(OS, DumpOpts);
> > >>   }
> > >> -
> > >> -  if ((DumpType & DIDT_DebugInfoDwo) &&
> > >> -      getNumDWOCompileUnits()) {
> > >> +  if (shouldDumpDWO(DIDT_DebugInfo, DObj->getInfoDWOSection().Data))
> > >> + {
> > >>     OS << "\n.debug_info.dwo contents:\n";
> > >>     for (const auto &DWOCU : dwo_compile_units())
> > >>       DWOCU->dump(OS, DumpOpts);
> > >>   }
> > >>
> > >> -  if ((DumpType & DIDT_DebugTypes) && getNumTypeUnits()) {
> > >> -    OS << "\n.debug_types contents:\n";
> > >> -    for (const auto &TUS : type_unit_sections())
> > >> -      for (const auto &TU : TUS)
> > >> -        TU->dump(OS, DumpOpts);
> > >> -  }
> > >> -
> > >> -  if ((DumpType & DIDT_DebugTypesDwo) &&
> > >> -      getNumDWOTypeUnits()) {
> > >> -    OS << "\n.debug_types.dwo contents:\n";
> > >> -    for (const auto &DWOTUS : dwo_type_unit_sections())
> > >> -      for (const auto &DWOTU : DWOTUS)
> > >> -        DWOTU->dump(OS, DumpOpts);
> > >> +  if ((DumpType & DIDT_DebugTypes)) {
> > >> +    if (Explicit || getNumTypeUnits()) {
> > >> +      OS << "\n.debug_types contents:\n";
> > >> +      for (const auto &TUS : type_unit_sections())
> > >> +        for (const auto &TU : TUS)
> > >> +          TU->dump(OS, DumpOpts);
> > >> +    }
> > >> +    if (ExplicitDWO || getNumDWOTypeUnits()) {
> > >> +      OS << "\n.debug_types.dwo contents:\n";
> > >> +      for (const auto &DWOTUS : dwo_type_unit_sections())
> > >> +        for (const auto &DWOTU : DWOTUS)
> > >> +          DWOTU->dump(OS, DumpOpts);
> > >> +    }
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugLoc) {
> > >> +  if (shouldDump(DIDT_DebugLoc, DObj->getLocSection().Data)) {
> > >>     OS << "\n.debug_loc contents:\n";
> > >>     getDebugLoc()->dump(OS, getRegisterInfo());
> > >>   }
> > >> -
> > >> -  if (DumpType & DIDT_DebugLocDwo) {
> > >> +  if (shouldDumpDWO(DIDT_DebugLoc, DObj->getLocDWOSection().Data)) {
> > >>     OS << "\n.debug_loc.dwo contents:\n";
> > >>     getDebugLocDWO()->dump(OS, getRegisterInfo());
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugFrames) {
> > >> +  if (shouldDump(DIDT_DebugFrames, DObj->getDebugFrameSection())) {
> > >>     OS << "\n.debug_frame contents:\n";
> > >>     getDebugFrame()->dump(OS);
> > >> -    if (DumpEH) {
> > >> -      OS << "\n.eh_frame contents:\n";
> > >> -      getEHFrame()->dump(OS);
> > >> -    }
> > >> +  }
> > >> +  if (DumpEH && !getEHFrame()->empty()) {
> > >> +    OS << "\n.eh_frame contents:\n";
> > >> +    getEHFrame()->dump(OS);
> > >>   }
> > >>
> > >>   if (DumpType & DIDT_DebugMacro) {
> > >> -    OS << "\n.debug_macinfo contents:\n";
> > >> -    getDebugMacro()->dump(OS);
> > >> +    if (Explicit || !getDebugMacro()->empty()) {
> > >> +      OS << "\n.debug_macinfo contents:\n";
> > >> +      getDebugMacro()->dump(OS);
> > >> +    }
> > >>   }
> > >>
> > >> -  uint32_t offset = 0;
> > >> -  if (DumpType & DIDT_DebugAranges) {
> > >> +  if (shouldDump(DIDT_DebugAranges, DObj->getARangeSection())) {
> > >>     OS << "\n.debug_aranges contents:\n";
> > >> +    uint32_t offset = 0;
> > >>     DataExtractor arangesData(DObj->getARangeSection(),
> isLittleEndian(),
> > 0);
> > >>     DWARFDebugArangeSet set;
> > >>     while (set.extract(arangesData, &offset)) @@ -304,7 +319,7 @@
> > >> void DWARFContext::dump(raw_ostream &OS,
> > >>   }
> > >>
> > >>   uint8_t savedAddressByteSize = 0;
> > >> -  if (DumpType & DIDT_DebugLine) {
> > >> +  if (shouldDump(DIDT_DebugLine, DObj->getLineSection().Data)) {
> > >>     OS << "\n.debug_line contents:\n";
> > >>     for (const auto &CU : compile_units()) {
> > >>       savedAddressByteSize = CU->getAddressByteSize(); @@ -322,17
> > >> +337,12 @@ void DWARFContext::dump(raw_ostream &OS,
> > >>     }
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugCUIndex) {
> > >> -    OS << "\n.debug_cu_index contents:\n";
> > >> -    getCUIndex().dump(OS);
> > >> -  }
> > >> -
> > >> -  if (DumpType & DIDT_DebugTUIndex) {
> > >> -    OS << "\n.debug_tu_index contents:\n";
> > >> -    getTUIndex().dump(OS);
> > >> +  // FIXME: This seems sketchy.
> > >> +  for (const auto &CU : compile_units()) {
> > >> +    savedAddressByteSize = CU->getAddressByteSize();
> > >> +    break;
> > >>   }
> > >> -
> > >> -  if (DumpType & DIDT_DebugLineDwo) {
> > >> +  if (shouldDumpDWO(DIDT_DebugLine, DObj->getLineDWOSection().Data))
> > >> + {
> > >>     OS << "\n.debug_line.dwo contents:\n";
> > >>     unsigned stmtOffset = 0;
> > >>     DWARFDataExtractor lineData(*DObj, DObj->getLineDWOSection(), @@
> > >> -344,22
> > >> +354,30 @@ void DWARFContext::dump(raw_ostream &OS,
> > >>     }
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugStr) {
> > >> +  if (shouldDump(DIDT_DebugCUIndex, DObj->getCUIndexSection())) {
> > >> +    OS << "\n.debug_cu_index contents:\n";
> > >> +    getCUIndex().dump(OS);
> > >> +  }
> > >> +
> > >> +  if (shouldDump(DIDT_DebugTUIndex, DObj->getTUIndexSection())) {
> > >> +    OS << "\n.debug_tu_index contents:\n";
> > >> +    getTUIndex().dump(OS);
> > >> +  }
> > >> +
> > >> +  if (shouldDump(DIDT_DebugStr, DObj->getStringSection())) {
> > >>     OS << "\n.debug_str contents:\n";
> > >>     DataExtractor strData(DObj->getStringSection(),
> isLittleEndian(), 0);
> > >> -    offset = 0;
> > >> +    uint32_t offset = 0;
> > >>     uint32_t strOffset = 0;
> > >>     while (const char *s = strData.getCStr(&offset)) {
> > >>       OS << format("0x%8.8x: \"%s\"\n", strOffset, s);
> > >>       strOffset = offset;
> > >>     }
> > >>   }
> > >> -
> > >> -  if ((DumpType & DIDT_DebugStrDwo) &&
> > >> -      !DObj->getStringDWOSection().empty()) {
> > >> +  if (shouldDumpDWO(DIDT_DebugStr, DObj->getStringDWOSection())) {
> > >>     OS << "\n.debug_str.dwo contents:\n";
> > >>     DataExtractor strDWOData(DObj->getStringDWOSection(),
> > >> isLittleEndian(), 0);
> > >> -    offset = 0;
> > >> +    uint32_t offset = 0;
> > >>     uint32_t strDWOOffset = 0;
> > >>     while (const char *s = strDWOData.getCStr(&offset)) {
> > >>       OS << format("0x%8.8x: \"%s\"\n", strDWOOffset, s); @@ -367,69
> > >> +385,68 @@ void DWARFContext::dump(raw_ostream &OS,
> > >>     }
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugRanges) {
> > >> +  if (shouldDump(DIDT_DebugRanges, DObj->getRangeSection().Data)) {
> > >>     OS << "\n.debug_ranges contents:\n";
> > >>     // In fact, different compile units may have different address
> byte
> > >> -    // sizes, but for simplicity we just use the address byte size
> of the
> > >> last
> > >> -    // compile unit (there is no easy and fast way to associate
> address
> > range
> > >> -    // list and the compile unit it describes).
> > >> +    // sizes, but for simplicity we just use the address byte size
> of the
> > >> +    // last compile unit (there is no easy and fast way to associate
> > address
> > >> +    // range list and the compile unit it describes).
> > >> +    // FIXME: savedAddressByteSize seems sketchy.
> > >>     DWARFDataExtractor rangesData(*DObj, DObj->getRangeSection(),
> > >>                                   isLittleEndian(),
> savedAddressByteSize);
> > >> -    offset = 0;
> > >> +    uint32_t offset = 0;
> > >>     DWARFDebugRangeList rangeList;
> > >>     while (rangeList.extract(rangesData, &offset))
> > >>       rangeList.dump(OS);
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_DebugPubnames)
> > >> +  if (shouldDump(DIDT_DebugPubnames, DObj->getPubNamesSection()))
> > >>     DWARFDebugPubTable(DObj->getPubNamesSection(), isLittleEndian(),
> false)
> > >>         .dump("debug_pubnames", OS);
> > >>
> > >> -  if (DumpType & DIDT_DebugPubtypes)
> > >> +  if (shouldDump(DIDT_DebugPubtypes, DObj->getPubTypesSection()))
> > >>     DWARFDebugPubTable(DObj->getPubTypesSection(), isLittleEndian(),
> false)
> > >>         .dump("debug_pubtypes", OS);
> > >>
> > >> -  if (DumpType & DIDT_DebugGnuPubnames)
> > >> +  if (shouldDump(DIDT_DebugGnuPubnames,
> > >> + DObj->getGnuPubNamesSection()))
> > >>     DWARFDebugPubTable(DObj->getGnuPubNamesSection(),
> isLittleEndian(),
> > >>                        true /* GnuStyle */)
> > >>         .dump("debug_gnu_pubnames", OS);
> > >>
> > >> -  if (DumpType & DIDT_DebugGnuPubtypes)
> > >> +  if (shouldDump(DIDT_DebugGnuPubtypes,
> > >> + DObj->getGnuPubTypesSection()))
> > >>     DWARFDebugPubTable(DObj->getGnuPubTypesSection(),
> isLittleEndian(),
> > >>                        true /* GnuStyle */)
> > >>         .dump("debug_gnu_pubtypes", OS);
> > >>
> > >> -  if (DumpType & DIDT_DebugStrOffsets)
> > >> +  if (shouldDump(DIDT_DebugStrOffsets,
> > >> + DObj->getStringOffsetSection().Data))
> > >>     dumpStringOffsetsSection(
> > >>         OS, "debug_str_offsets", *DObj, DObj->getStringOffsetSection()
> ,
> > >>         DObj->getStringSection(), isLittleEndian(), getMaxVersion());
> > >> -
> > >> -  if (DumpType & DIDT_DebugStrOffsetsDwo) {
> > >> +  if (shouldDumpDWO(DIDT_DebugStrOffsets,
> > >> +                    DObj->getStringOffsetDWOSection().Data))
> > >>     dumpStringOffsetsSection(
> > >>         OS, "debug_str_offsets.dwo", *DObj, DObj-
> > >>> getStringOffsetDWOSection(),
> > >>         DObj->getStringDWOSection(), isLittleEndian(),
> > >> getMaxVersion());
> > >> -  }
> > >>
> > >> -  if ((DumpType & DIDT_GdbIndex) &&
> > >> -      !DObj->getGdbIndexSection().empty()) {
> > >> +  if (shouldDump(DIDT_GdbIndex, DObj->getGdbIndexSection())) {
> > >>     OS << "\n.gnu_index contents:\n";
> > >>     getGdbIndex().dump(OS);
> > >>   }
> > >>
> > >> -  if (DumpType & DIDT_AppleNames)
> > >> +  if (shouldDump(DIDT_AppleNames,
> > >> + DObj->getAppleNamesSection().Data))
> > >>     dumpAccelSection(OS, "apple_names", *DObj, DObj-
> > >getAppleNamesSection(),
> > >>                      DObj->getStringSection(), isLittleEndian());
> > >>
> > >> -  if (DumpType & DIDT_AppleTypes)
> > >> +  if (shouldDump(DIDT_AppleTypes,
> > >> + DObj->getAppleTypesSection().Data))
> > >>     dumpAccelSection(OS, "apple_types", *DObj, DObj-
> > >getAppleTypesSection(),
> > >>                      DObj->getStringSection(), isLittleEndian());
> > >>
> > >> -  if (DumpType & DIDT_AppleNamespaces)
> > >> +  if (shouldDump(DIDT_AppleNamespaces,
> > >> + DObj->getAppleNamespacesSection().Data))
> > >>     dumpAccelSection(OS, "apple_namespaces", *DObj,
> > >>                      DObj->getAppleNamespacesSection(),
> > >>                      DObj->getStringSection(), isLittleEndian());
> > >>
> > >> -  if (DumpType & DIDT_AppleObjC)
> > >> +  if (shouldDump(DIDT_AppleObjC, DObj->getAppleObjCSection().Data))
> > >>     dumpAccelSection(OS, "apple_objc", *DObj,
> DObj->getAppleObjCSection(),
> > >>                      DObj->getStringSection(), isLittleEndian());  }
> > >> @@ -
> > >> 461,7 +478,7 @@ DWARFDie DWARFContext::getDIEForOffset(u
> > >>   return DWARFDie();
> > >> }
> > >>
> > >> -bool DWARFContext::verify(raw_ostream &OS, uint64_t DumpType,
> > >> +bool DWARFContext::verify(raw_ostream &OS, unsigned DumpType,
> > >>                           DIDumpOptions DumpOpts) {
> > >>   bool Success = true;
> > >>   DWARFVerifier verifier(OS, *this, DumpOpts);
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/Inputs/gmlt.ll
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/DebugInfo/Inputs/gmlt.ll?rev=313207&r1=313206
> > >> &r2=31320
> > >> 7&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/Inputs/gmlt.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/Inputs/gmlt.ll Wed Sep 13 15:09:01 2017
> > >> @@ -87,12 +87,8 @@
> > >>
> > >>
> > >> ; Check that we don't emit any pubnames or pubtypes under -gmlt -;
> CHECK:
> > >> .debug_pubnames contents:
> > >> -; CHECK-NOT: Offset
> > >> -
> > >> -; CHECK: .debug_pubtypes contents:
> > >> -; CHECK-NOT: Offset
> > >> -
> > >> +; CHECK-NOT: .debug_pubnames contents:
> > >> +; CHECK-NOT: .debug_pubtypes contents:
> > >> ; CHECK: .apple{{.*}} contents:
> > >>
> > >> ; Function Attrs: nounwind uwtable
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/fission-inline.ll
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/DebugInfo/X86/fission-
> > >> inline.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/fission-inline.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/fission-inline.ll Wed Sep 13
> > >> +++ 15:09:01
> > >> +++ 2017
> > >> @@ -71,6 +71,7 @@
> > >> ; CHECK:     DW_AT_call_file
> > >> ; CHECK-NEXT:     DW_AT_call_line {{.*}} (18)
> > >> ; CHECK-NOT: DW_
> > >> +; CHECK: .debug_info.dwo contents:
> > >>
> > >> ; RELOCS-NOT: RELOCATION RECORDS FOR [.rela.debug_ranges]
> > >>
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/fission-no-inlining.ll
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/DebugInfo/X86/fission-no-
> > >> inlining.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/fission-no-inlining.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/fission-no-inlining.ll Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -1,6 +1,8 @@
> > >> ; RUN: llc -split-dwarf-file=foo.dwo -O0 < %s
> > >> -mtriple=x86_64-unknown-linux- gnu -filetype=obj | llvm-dwarfdump
> > >> -debug-info - | FileCheck %s
> > >>
> > >> +; CHECK: .debug_info contents:
> > >> ; CHECK-NOT: DW_TAG_subprogram
> > >> +; CHECK: contents:
> > >>
> > >> ; IR generated from the following source:
> > >> ; void f1();
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/fission-ranges.ll
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/DebugInfo/X86/fission-
> > >> ranges.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/fission-ranges.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/fission-ranges.ll Wed Sep 13
> > >> +++ 15:09:01
> > >> +++ 2017
> > >> @@ -17,7 +17,7 @@
> > >> ; CHECK: DW_AT_location [DW_FORM_sec_offset]   ([[B:0x[0-9a-z]*]]
> > >> ; CHECK: DW_AT_location [DW_FORM_sec_offset]   ([[D:0x[0-9a-z]*]]
> > >> ; CHECK: DW_AT_ranges [DW_FORM_sec_offset]   (0x00000000
> > >> -; CHECK: .debug_loc contents:
> > >> +; CHECK-NOT: .debug_loc contents:
> > >> ; CHECK-NOT: Beginning address offset ; CHECK: .debug_loc.dwo
> > >> contents:
> > >>
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/generate-odr-hash.ll
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/DebugInfo/X86/generate-odr-
> > >> hash.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/generate-odr-hash.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/generate-odr-hash.ll Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -74,8 +74,7 @@
> > >> ; CHECK-NEXT: DW_AT_signature {{.*}} (0xfd756cee88f8a118)
> > >>
> > >> ; SINGLE-LABEL: .debug_types contents:
> > >> -; FISSION-NOT: .debug_types contents:
> > >> -; FISSION-LABEL: .debug_types.dwo contents:
> > >> +; FISSION: .debug_types.dwo contents:
> > >>
> > >> ; Check that we generate a hash for bar and the value.
> > >> ; CHECK-NOT: type_signature
> > >> @@ -127,7 +126,8 @@
> > >> ; CHECK: file_names{{.*}} bar.cpp
> > >> ; CHECK-NOT: file_names[
> > >>
> > >> -; CHECK-LABEL: .debug_line.dwo contents:
> > >> +; FISSION: .debug_line.dwo contents:
> > >> +; CHECK-NOT: .debug_line.dwo contents:
> > >> ; FISSION: Line table prologue
> > >> ; FISSION: opcode_base: 1
> > >> ; FISSION-NOT: standard_opcode_lengths
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/gnu-public-names-gmlt.ll
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/X86/gnu
> > >> - public-names-gmlt.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/gnu-public-names-gmlt.ll (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/gnu-public-names-gmlt.ll Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -24,14 +24,10 @@
> > >> ; GPUB-NEXT: length = 0x0000000e version = 0x0002 unit_offset =
> > >> 0x00000000  ;
> > >> GPUB-NEXT: Name
> > >>
> > >> -; NONE: .debug_pubnames contents:
> > >> -; NONE: {{^$}}
> > >> -; NONE: .debug_pubtypes contents:
> > >> -; NONE: {{^$}}
> > >> -; NONE: .debug_gnu_pubnames contents:
> > >> -; NONE: {{^$}}
> > >> -; NONE: .debug_gnu_pubtypes contents:
> > >> -; NONE: {{^$}}
> > >> +; NONE-NOT: .debug_pubnames contents:
> > >> +; NONE-NOT: .debug_pubtypes contents:
> > >> +; NONE-NOT: .debug_gnu_pubnames contents:
> > >> +; NONE-NOT: .debug_gnu_pubtypes contents:
> > >>
> > >>
> > >> ; Function Attrs: noinline uwtable
> > >>
> > >> Modified:
> > >> llvm/trunk/test/DebugInfo/X86/split-dwarf-cross-unit-reference.ll
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/X86/spl
> > >> it-
> > >> dwarf-cross-unit-reference.ll?rev=313207&r1=313206&r2=313207&view=dif
> > >> f
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/split-dwarf-cross-unit-reference.ll
> > >> (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/split-dwarf-cross-unit-reference.ll
> > >> +++ Wed Sep 13 15:09:01 2017
> > >> @@ -1,11 +1,11 @@
> > >> ; RUN: llc -mtriple=x86_64-linux -split-dwarf-cross-cu-references
> > >> -split- dwarf-file=foo.dwo -filetype=obj -o %t < %s  ; RUN:
> > >> llvm-objdump -r %t | FileCheck %s -; RUN: llvm-dwarfdump -v
> > >> -debug-info-dwo %t | FileCheck --check- prefix=ALL
> > >> --check-prefix=INFO --check-prefix=DWO --check-prefix=CROSS %s
> > >> +; RUN: llvm-dwarfdump -v -debug-info %t | FileCheck
> > >> +--check-prefix=ALL --check-prefix=INFO --check-prefix=DWO
> > >> +--check-prefix=CROSS %s
> > >> ; RUN: llvm-dwarfdump -v -debug-info %t | FileCheck
> > >> --check-prefix=ALL -- check-prefix=INFO %s
> > >>
> > >> ; RUN: llc -mtriple=x86_64-linux -split-dwarf-file=foo.dwo
> > >> -filetype=obj -o %t < %s  ; RUN: llvm-objdump -r %t | FileCheck %s -;
> > >> RUN: llvm-dwarfdump -v - debug-info-dwo %t | FileCheck
> > >> --check-prefix=ALL --check-prefix=DWO --check- prefix=NOCROSS %s
> > >> +; RUN: llvm-dwarfdump -v -debug-info %t | FileCheck
> > >> +--check-prefix=ALL --check-prefix=DWO --check-prefix=NOCROSS %s
> > >> ; RUN: llvm-dwarfdump -v -debug-info %t | FileCheck
> > >> --check-prefix=ALL -- check-prefix=INFO %s
> > >>
> > >> ; Testing cross-CU references for types, subprograms, and variables
> > >> @@ -42,6
> > >> +42,7 @@  ; * debug_info.dwo contains duplicate types, abstract
> > >> +subprograms
> > >> and abstract
> > >> ;   variables otherwise to avoid the need for cross-cu references
> > >>
> > >> +; DWO: .debug_info.dwo contents:
> > >> ; CHECK-NOT: .rel{{a?}}.debug_info.dwo ; CHECK: RELOCATION RECORDS
> > >> FOR [.rel{{a?}}.debug_info]:
> > >> ; CHECK-NOT: RELOCATION RECORDS
> > >>
> > >> Modified:
> > >> llvm/trunk/test/DebugInfo/X86/split-dwarf-multiple-cu-hash.ll
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/X86/spl
> > >> it-
> > >> dwarf-multiple-cu-hash.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/split-dwarf-multiple-cu-hash.ll
> > >> (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/split-dwarf-multiple-cu-hash.ll Wed
> > >> +++ Sep 13 15:09:01 2017
> > >> @@ -3,8 +3,10 @@
> > >> ; RUN: %llc_dwarf -split-dwarf-file=bar.dwo  %s -filetype=obj -o
> > >> %t/b.o  ;
> > >> RUN: llvm-dwarfdump -debug-info %t/a.o %t/b.o | FileCheck %s
> > >>
> > >> +; CHECK: .debug_info contents:
> > >> ; CHECK: dwo_id {{.*}}([[HASH:.*]])
> > >> ; CHECK-NOT: dwo_id {{.*}}([[HASH]])
> > >> +; CHECK: .debug_info.dwo contents:
> > >>
> > >> target triple = "x86_64-pc-linux"
> > >>
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/X86/split-dwarf-omit-empty.ll
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/X86/spl
> > >> it- dwarf-omit-empty.ll?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/X86/split-dwarf-omit-empty.ll
> > >> (original)
> > >> +++ llvm/trunk/test/DebugInfo/X86/split-dwarf-omit-empty.ll Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -15,8 +15,10 @@
> > >> ; will be emitted. This emulates something more like the
> > >> available_externally ; import performed by ThinLTO.
> > >>
> > >> +; CHECK: .debug_info contents:
> > >> ; CHECK: Compile Unit
> > >> ; CHECK-NOT: Compile Unit
> > >> +; CHECK: .debug_info.dwo contents:
> > >>
> > >> target triple = "x86_64-pc-linux"
> > >>
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/dwarfdump-dump-flags.test
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/dwarfdu
> > >> mp- dump-flags.test?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/dwarfdump-dump-flags.test (original)
> > >> +++ llvm/trunk/test/DebugInfo/dwarfdump-dump-flags.test Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -6,10 +6,11 @@
> > >> ; RUN: llvm-dwarfdump %p/Inputs/dwarfdump-test.macho-i386.o
> > >> --debug-ranges | FileCheck %s -check-prefix DUMP_RANGES
> > >>
> > >> ; DUMP_ALL: .debug_info
> > >> -; DUMP_ALL: .debug_ranges
> > >> +; DUMP_ALL: .debug_str
> > >>
> > >> ; DUMP_INFO: .debug_info
> > >> ; DUMP_INFO-NOT: .debug_ranges
> > >> +; DUMP_INFO-NOT: .debug_str
> > >>
> > >> ; DUMP_RANGES-NOT: .debug_info
> > >> ; DUMP_RANGES: .debug_ranges
> > >>
> > >> Modified: llvm/trunk/test/DebugInfo/dwarfdump-dwp.test
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/dwarfdu
> > >> mp- dwp.test?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/DebugInfo/dwarfdump-dwp.test (original)
> > >> +++ llvm/trunk/test/DebugInfo/dwarfdump-dwp.test Wed Sep 13 15:09:01
> > >> +++ 2017
> > >> @@ -9,6 +9,7 @@ RUN: llvm-dwarfdump -v %p/Inputs/dwarfdu
> > >> ;   bar b() {
> > >> ;   }
> > >>
> > >> +; CHECK-NOT: .debug_info contents:
> > >> ; CHECK-LABEL: .debug_info.dwo contents:
> > >> ; CHECK: Compile Unit
> > >>
> > >>
> > >> Modified:
> > >> llvm/trunk/test/MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/ARM/dwarf-asm-
> > >> multiple-sections-dwarf-2.s?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s
> > >> (original)
> > >> +++ llvm/trunk/test/MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s Wed
> > >> +++ Sep
> > >> +++ 13 15:09:01 2017
> > >> @@ -45,9 +45,8 @@ b:
> > >> // DWARF-NEXT: 0x0000000000000004     11      0      1   0   0 is_stmt
> > >> end_sequence
> > >>
> > >>
> > >> -// DWARF: .debug_ranges contents:
> > >> -// DWARF-NOT: {{0-9a-f}}
> > >> -// DWARF: .debug_pubnames contents:
> > >> +// DWARF-NOT: .debug_ranges contents:
> > >> +// DWARF-NOT: .debug_pubnames contents:
> > >>
> > >>
> > >> // RELOC: RELOCATION RECORDS FOR [.rel.debug_info]:
> > >>
> > >> Modified: llvm/trunk/test/MC/ARM/dwarf-asm-no-code.s
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/ARM/dwarf-asm-
> > >> no- code.s?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/MC/ARM/dwarf-asm-no-code.s (original)
> > >> +++ llvm/trunk/test/MC/ARM/dwarf-asm-no-code.s Wed Sep 13 15:09:01
> > >> +++ 2017
> > >> @@ -8,18 +8,10 @@
> > >> a:
> > >> .long 42
> > >>
> > >> -// DWARF: .debug_abbrev contents:
> > >> -// DWARF-NEXT: < EMPTY >
> > >> -
> > >> -// DWARF: .debug_info contents:
> > >> -
> > >> -// DWARF: .debug_aranges contents:
> > >> -
> > >> +// DWARF: ELF32-arm-little
> > >> +// DWARF-NOT: contents:
> > >> // DWARF: .debug_line contents:
> > >>
> > >> -// DWARF: .debug_ranges contents:
> > >> -
> > >> -
> > >> // RELOC-NOT: RELOCATION RECORDS FOR [.rel.debug_info]:
> > >>
> > >> // RELOC-NOT: RELOCATION RECORDS FOR [.rel.debug_ranges]:
> > >>
> > >> Modified: llvm/trunk/test/MC/ARM/dwarf-asm-nonstandard-section.s
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/ARM/dwarf-asm-
> > >> nonstandard-section.s?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/MC/ARM/dwarf-asm-nonstandard-section.s (original)
> > >> +++ llvm/trunk/test/MC/ARM/dwarf-asm-nonstandard-section.s Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -37,9 +37,8 @@ b:
> > >> // DWARF-NEXT: 0x0000000000000004      7      0      1   0   0
> is_stmt
> > >> end_sequence
> > >>
> > >>
> > >> -// DWARF: .debug_ranges contents:
> > >> -// DWARF-NOT: {{0-9a-f}}
> > >> -// DWARF: .debug_pubnames contents:
> > >> +// DWARF-NOT: .debug_ranges contents:
> > >> +// DWARF-NOT: .debug_pubnames contents:
> > >>
> > >>
> > >>
> > >>
> > >> Modified: llvm/trunk/test/MC/ARM/dwarf-asm-single-section.s
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/ARM/dwarf-asm-
> > >> single-section.s?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/MC/ARM/dwarf-asm-single-section.s (original)
> > >> +++ llvm/trunk/test/MC/ARM/dwarf-asm-single-section.s Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -37,9 +37,8 @@ a:
> > >> // DWARF-NEXT: 0x0000000000000004      7      0      1   0   0 is_stmt
> > >> end_sequence
> > >>
> > >>
> > >> -// DWARF: .debug_ranges contents:
> > >> -// DWARF-NOT: {{0-9a-f}}
> > >> -// DWARF: .debug_pubnames contents:
> > >> +// DWARF-NOT: .debug_ranges contents:
> > >> +// DWARF-NOT: .debug_pubnames contents:
> > >>
> > >>
> > >> // RELOC: RELOCATION RECORDS FOR [.rel.debug_info]:
> > >>
> > >> Modified: llvm/trunk/test/tools/dsymutil/X86/basic-linking-x86.test
> > >> URL: http://llvm.org/viewvc/llvm-
> > >> project/llvm/trunk/test/tools/dsymutil/X86/basic-linking-
> > >> x86.test?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/tools/dsymutil/X86/basic-linking-x86.test
> > >> (original)
> > >> +++ llvm/trunk/test/tools/dsymutil/X86/basic-linking-x86.test Wed Sep
> > >> +++ 13
> > >> +++ 15:09:01 2017
> > >> @@ -140,8 +140,7 @@ CHECK:      DW_AT_frame_base [DW_FORM_bl
> > >>
> > >> CHECK:    NULL
> > >>
> > >> -CHECK: .debug_loc contents
> > >> -CHECK-NOT: Location
> > >> +CHECK-NOT: .debug_loc contents
> > >>
> > >> CHECK:.debug_aranges contents:
> > >> CHECK-NEXT:Address Range Header: length = 0x0000002c, version =
> > >> 0x0002, cu_offset = 0x00000000, addr_size = 0x08, seg_size = 0x00
> > >>
> > >> Modified: llvm/trunk/test/tools/llvm-dwp/X86/empty.test
> > >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-
> > >> dwp/X86/empty.test?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/tools/llvm-dwp/X86/empty.test (original)
> > >> +++ llvm/trunk/test/tools/llvm-dwp/X86/empty.test Wed Sep 13 15:09:01
> > >> +++ 2017
> > >> @@ -1,8 +1,8 @@
> > >> RUN: llvm-dwp %p/../Inputs/empty.dwo -o %t
> > >> RUN: llvm-dwarfdump -v %t | FileCheck %s
> > >>
> > >> -CHECK-LABEL: .debug_cu_index
> > >> +CHECK: file format
> > >> +CHECK-NOT: .debug_cu_index
> > >> CHECK-NOT: version
> > >> -CHECK-LABEL: .debug_tu_index
> > >> +CHECK-NOT: .debug_tu_index
> > >> CHECK-NOT: version
> > >> -CHECK: .debug_
> > >>
> > >> Modified: llvm/trunk/test/tools/llvm-dwp/X86/simple.test
> > >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-
> > >> dwp/X86/simple.test?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/test/tools/llvm-dwp/X86/simple.test (original)
> > >> +++ llvm/trunk/test/tools/llvm-dwp/X86/simple.test Wed Sep 13
> > >> +++ 15:09:01
> > >> +++ 2017
> > >> @@ -76,8 +76,7 @@ TYPES:     3 [[DWOB]]  {{\[}}[[BOFF]], [
> > >> NOTYP:     3 [[DWOA]]  {{\[}}[[AOFF]], [[BOFF]]) [0x0000[[AAOFF]],
> > >> 0x0000[[BAOFF]]) [0x00000000, 0x00000011) [0x00000000, 0x00000010)
> > >> NOTYP:     4 [[DWOB]]  {{\[}}[[BOFF]], [[XOFF]]) [0x0000[[BAOFF]],
> > >> 0x00000075)      [0x00000011, 0x00000022) [0x00000010, 0x00000024)
> > >>
> > >> -CHECK-LABEL: .debug_tu_index contents:
> > >> -NOTYP-NOT: Index
> > >> +CHECK-NOT: .debug_tu_index contents:
> > >> TYPES: Index Signature  TYPES                           ABBREV
> > >> LINE                     STR_OFFSETS
> > >> TYPES:     1 [[FOOSIG]] {{\[}}[[FOOUOFF]], [[BARUOFF]])
> [0x0000[[AAOFF]],
> > >> 0x0000[[BAOFF]]) [0x00000000, 0x0000001a) [0x00000000, 0x00000010)
> > >> TYPES:     4 [[BARSIG]] {{\[}}[[BARUOFF]], [[XUOFF]])
>  [0x0000[[BAOFF]],
> > >> 0x00000099)      [0x0000001a, 0x00000034) [0x00000010, 0x00000024)
> > >>
> > >> Modified: llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
> > >> URL:
> > >> http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-dwarfdump/l
> > >> lvm- dwarfdump.cpp?rev=313207&r1=313206&r2=313207&view=diff
> > >> =====================================================================
> > >> =========
> > >> --- llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp (original)
> > >> +++ llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp Wed Sep 13
> > >> +++ 15:09:01 2017
> > >> @@ -53,7 +53,7 @@ static opt<bool> DumpAll("all", desc("Du
> > >>                          cat(SectionCategory));  static alias
> > >> DumpAllAlias("a", desc("Alias for -all"), aliasopt(DumpAll));
> > >>
> > >> -static uint64_t DumpType = DIDT_Null;
> > >> +static unsigned DumpType = DIDT_Null;
> > >> #define HANDLE_DWARF_SECTION(ENUM_NAME, ELF_NAME, CMDLINE_NAME) \
> > >>   static opt<bool> Dump##ENUM_NAME(CMDLINE_NAME, \
> > >>                                    desc("Dump the " ELF_NAME "
> > >> section"), \ @@ -99,8 +99,7 @@ static void DumpObjectFile(ObjectFile
> > >> &O
> > >>                         Filename.str() + ": ");
> > >>   // The UUID dump already contains all the same information.
> > >>   if (!(DumpType & DIDT_UUID) || DumpType == DIDT_All)
> > >> -    outs() << Filename << ":\tfile format " <<
> Obj.getFileFormatName()
> > >> -           << "\n\n";
> > >> +    outs() << Filename << ":\tfile format " <<
> > >> + Obj.getFileFormatName() << '\n';
> > >>
> > >>   // Dump the complete DWARF structure.
> > >>   DICtx->dump(outs(), GetDumpOpts());
> > >>
> > >>
> > >> _______________________________________________
> > >> llvm-commits mailing list
> > >> llvm-commits at lists.llvm.org
> > >> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170914/a59b9d45/attachment-0001.html>


More information about the llvm-commits mailing list