From d42f155408bfea0ff23d2408725753c5778bf544 Mon Sep 17 00:00:00 2001
From: Ali Tamur <tamur@google.com>
Date: Tue, 26 Mar 2019 18:53:23 +0000
Subject: [PATCH] [llvm] Reapply "Prevent duplicate files in debug line header
 in dwarf 5."

Reapply rL356941 after regenerating the object file in the failing test
llvm/test/tools/llvm-objdump/embedded-source.test from source.

Original commit message:

[llvm] Prevent duplicate files in debug line header in dwarf 5.

Motivation: In previous dwarf versions, file name indexes started from 1, and
the primary source file was not explicit. Dwarf 5 standard (6.2.4) prescribes
the primary source file to be explicitly given an entry with an index number 0.

The current implementation honors the specification by just duplicating the
main source file, once with index number 0, and later maybe with another
index number. While this is compliant with the letter of the standard, the
duplication causes problems for consumers of this information such as lldb.
(Some files are duplicated, where only some of them have a line table although
all refer to the same file)

With this change, dwarf 5 debug line section files always start from 0, and
the zeroth entry is not duplicated whenever possible. This requires different
handling of dwarf 4 and dwarf 5 during generation (e.g. when a function returns
an index zero for a file name, it signals an error in dwarf 4, but not in dwarf 5)
However, I think the minor complication is worth it, because it enables all
consumers (lldb, gdb, dwarfdump, objdump, and so on) to treat all files in the
file name list homogenously.

Tags: #llvm, #debug-info

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@357018 91177308-0d34-0410-b5e6-96231b3b80d8
---
 include/llvm/DebugInfo/DWARF/DWARFDebugLine.h |   2 ++
 include/llvm/MC/MCContext.h                   |   2 +-
 include/llvm/MC/MCDwarf.h                     |  18 +++++++++++++++-
 lib/CodeGen/AsmPrinter/DwarfCompileUnit.h     |   1 +
 lib/CodeGen/AsmPrinter/DwarfDebug.cpp         |   1 +
 lib/CodeGen/AsmPrinter/DwarfUnit.cpp          |   1 -
 lib/DebugInfo/DWARF/DWARFDebugLine.cpp        |  20 +++++++++++++++---
 lib/MC/MCContext.cpp                          |   5 +++--
 lib/MC/MCDwarf.cpp                            |  11 ++++++++++
 test/MC/ARM/dwarf-asm-multiple-sections.s     |  17 +++++++++------
 test/MC/ELF/debug-mixed-md5.ll                |   3 +--
 test/MC/ELF/dwarf-file0.s                     |   6 ++----
 .../tools/llvm-objdump/Inputs/embedded-source | Bin 9936 -> 9448 bytes
 .../X86/function-sections-line-numbers.s      |   9 ++++----
 14 files changed, 70 insertions(+), 26 deletions(-)

diff --git a/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h b/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h
index 1f36e0daed0..6247c31d717 100644
--- a/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h
+++ b/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h
@@ -275,6 +275,8 @@ public:
     SequenceVector Sequences;
 
   private:
+    const llvm::DWARFDebugLine::FileNameEntry &
+    getFileNameEntry(uint64_t Index) const;
     uint32_t findRowInSeq(const DWARFDebugLine::Sequence &Seq,
                           object::SectionedAddress Address) const;
     Optional<StringRef>
diff --git a/include/llvm/MC/MCContext.h b/include/llvm/MC/MCContext.h
index c87df5a82ce..6beb6463964 100644
--- a/include/llvm/MC/MCContext.h
+++ b/include/llvm/MC/MCContext.h
@@ -521,7 +521,7 @@ namespace llvm {
     }
 
     MCDwarfLineTable &getMCDwarfLineTable(unsigned CUID) {
-      return MCDwarfLineTablesCUMap[CUID];
+      return MCDwarfLineTablesCUMap.emplace(CUID, DwarfVersion).first->second;
     }
 
     const MCDwarfLineTable &getMCDwarfLineTable(unsigned CUID) const {
diff --git a/include/llvm/MC/MCDwarf.h b/include/llvm/MC/MCDwarf.h
index 991ad525f89..70624d0a4df 100644
--- a/include/llvm/MC/MCDwarf.h
+++ b/include/llvm/MC/MCDwarf.h
@@ -218,9 +218,11 @@ struct MCDwarfLineTableHeader {
 private:
   bool HasAllMD5 = true;
   bool HasAnyMD5 = false;
+  unsigned DwarfVersion;
 
 public:
-  MCDwarfLineTableHeader() = default;
+  explicit MCDwarfLineTableHeader(unsigned DwarfVersion) :
+    DwarfVersion(DwarfVersion) {}
 
   Expected<unsigned> tryGetFile(StringRef &Directory, StringRef &FileName,
                                 MD5::MD5Result *Checksum,
@@ -245,6 +247,17 @@ public:
     return MCDwarfFiles.empty() || (HasAllMD5 == HasAnyMD5);
   }
 
+  void setRootFile(StringRef Directory, StringRef FileName,
+                   MD5::MD5Result *Checksum, Optional<StringRef> Source) {
+    CompilationDir = Directory;
+    RootFile.Name = FileName;
+    RootFile.DirIndex = 0;
+    RootFile.Checksum = Checksum;
+    RootFile.Source = Source;
+    trackMD5Usage(Checksum);
+    HasSource = Source.hasValue();
+  }
+
 private:
   void emitV2FileDirTables(MCStreamer *MCOS) const;
   void emitV5FileDirTables(MCStreamer *MCOS, Optional<MCDwarfLineStr> &LineStr,
@@ -255,6 +268,8 @@ class MCDwarfDwoLineTable {
   MCDwarfLineTableHeader Header;
 
 public:
+  MCDwarfDwoLineTable(unsigned DwarfVersion) : Header(DwarfVersion) {}
+
   void maybeSetRootFile(StringRef Directory, StringRef FileName,
                         MD5::MD5Result *Checksum, Optional<StringRef> Source) {
     if (!Header.RootFile.Name.empty())
@@ -282,6 +297,7 @@ class MCDwarfLineTable {
   MCLineSection MCLineSections;
 
 public:
+  MCDwarfLineTable(unsigned DwarfVersion) : Header(DwarfVersion) {}
   // This emits the Dwarf file and the line tables for all Compile Units.
   static void Emit(MCObjectStreamer *MCOS, MCDwarfLineTableParams Params);
 
diff --git a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h
index d9addb52376..683d3245e56 100644
--- a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h
+++ b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h
@@ -101,6 +101,7 @@ class DwarfCompileUnit final : public DwarfUnit {
   }
 
 public:
+  unsigned getDwarfVersion() const { return DD->getDwarfVersion(); }
   DwarfCompileUnit(unsigned UID, const DICompileUnit *Node, AsmPrinter *A,
                    DwarfDebug *DW, DwarfFile *DWU);
 
diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
index 0e0a5b24864..226014cf0bb 100644
--- a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
+++ b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
@@ -319,6 +319,7 @@ DwarfDebug::DwarfDebug(AsmPrinter *A, Module *M)
     : DebugHandlerBase(A), DebugLocs(A->OutStreamer->isVerboseAsm()),
       InfoHolder(A, "info_string", DIEValueAllocator),
       SkeletonHolder(A, "skel_string", DIEValueAllocator),
+      SplitTypeUnitFileTable(A->getDwarfVersion()),
       IsDarwin(A->TM.getTargetTriple().isOSDarwin()) {
   const Triple &TT = Asm->TM.getTargetTriple();
 
diff --git a/lib/CodeGen/AsmPrinter/DwarfUnit.cpp b/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
index 6e547469c24..5d8e236a6b3 100644
--- a/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
+++ b/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
@@ -397,7 +397,6 @@ void DwarfUnit::addSourceLine(DIE &Die, unsigned Line, const DIFile *File) {
     return;
 
   unsigned FileID = getOrCreateSourceID(File);
-  assert(FileID && "Invalid file id");
   addUInt(Die, dwarf::DW_AT_decl_file, None, FileID);
   addUInt(Die, dwarf::DW_AT_decl_line, None, Line);
 }
diff --git a/lib/DebugInfo/DWARF/DWARFDebugLine.cpp b/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
index 53420187773..6ad06a8553e 100644
--- a/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
+++ b/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
@@ -1017,14 +1017,28 @@ bool DWARFDebugLine::LineTable::lookupAddressRangeImpl(
 }
 
 bool DWARFDebugLine::LineTable::hasFileAtIndex(uint64_t FileIndex) const {
-  return FileIndex != 0 && FileIndex <= Prologue.FileNames.size();
+  uint16_t DwarfVersion = Prologue.getVersion();
+  assert(DwarfVersion != 0 && "LineTable has no dwarf version information");
+  if (DwarfVersion >= 5)
+    return FileIndex < Prologue.FileNames.size();
+  else
+    return FileIndex != 0 && FileIndex <= Prologue.FileNames.size();
+}
+const llvm::DWARFDebugLine::FileNameEntry &
+DWARFDebugLine::LineTable::getFileNameEntry(uint64_t Index) const {
+  uint16_t DwarfVersion = Prologue.getVersion();
+  assert(DwarfVersion != 0 && "LineTable has no dwarf version information");
+  if (DwarfVersion >= 5)
+    return Prologue.FileNames[Index];
+  else
+    return Prologue.FileNames[Index - 1];
 }
 
 Optional<StringRef> DWARFDebugLine::LineTable::getSourceByIndex(uint64_t FileIndex,
                                                                 FileLineInfoKind Kind) const {
   if (Kind == FileLineInfoKind::None || !hasFileAtIndex(FileIndex))
     return None;
-  const FileNameEntry &Entry = Prologue.FileNames[FileIndex - 1];
+  const FileNameEntry &Entry = getFileNameEntry(FileIndex);
   if (Optional<const char *> source = Entry.Source.getAsCString())
     return StringRef(*source);
   return None;
@@ -1044,7 +1058,7 @@ bool DWARFDebugLine::LineTable::getFileNameByIndex(uint64_t FileIndex,
                                                    std::string &Result) const {
   if (Kind == FileLineInfoKind::None || !hasFileAtIndex(FileIndex))
     return false;
-  const FileNameEntry &Entry = Prologue.FileNames[FileIndex - 1];
+  const FileNameEntry &Entry = getFileNameEntry(FileIndex);
   StringRef FileName = Entry.Name.getAsCString().getValue();
   if (Kind != FileLineInfoKind::AbsoluteFilePath ||
       isPathAbsoluteOnWindowsOrPosix(FileName)) {
diff --git a/lib/MC/MCContext.cpp b/lib/MC/MCContext.cpp
index 49fad131a2c..348436f4bd0 100644
--- a/lib/MC/MCContext.cpp
+++ b/lib/MC/MCContext.cpp
@@ -603,7 +603,8 @@ Expected<unsigned> MCContext::getDwarfFile(StringRef Directory,
                                            MD5::MD5Result *Checksum,
                                            Optional<StringRef> Source,
                                            unsigned CUID) {
-  MCDwarfLineTable &Table = MCDwarfLineTablesCUMap[CUID];
+  MCDwarfLineTable &Table =
+      MCDwarfLineTablesCUMap.emplace(CUID, DwarfVersion).first->second;
   return Table.tryGetFile(Directory, FileName, Checksum, Source, FileNumber);
 }
 
@@ -612,7 +613,7 @@ Expected<unsigned> MCContext::getDwarfFile(StringRef Directory,
 bool MCContext::isValidDwarfFileNumber(unsigned FileNumber, unsigned CUID) {
   const MCDwarfLineTable &LineTable = getMCDwarfLineTable(CUID);
   if (FileNumber == 0)
-    return getDwarfVersion() >= 5 && LineTable.hasRootFile();
+    return getDwarfVersion() >= 5;
   if (FileNumber >= LineTable.getMCDwarfFiles().size())
     return false;
 
diff --git a/lib/MC/MCDwarf.cpp b/lib/MC/MCDwarf.cpp
index 83b6b4f1aa3..73b4d4bcd19 100644
--- a/lib/MC/MCDwarf.cpp
+++ b/lib/MC/MCDwarf.cpp
@@ -542,6 +542,15 @@ Expected<unsigned> MCDwarfLineTable::tryGetFile(StringRef &Directory,
   return Header.tryGetFile(Directory, FileName, Checksum, Source, FileNumber);
 }
 
+bool isRootFile(const MCDwarfFile &RootFile, StringRef &Directory,
+                StringRef &FileName, MD5::MD5Result *Checksum) {
+  if (RootFile.Name.empty() || RootFile.Name != FileName.data())
+    return false;
+  if (!RootFile.Checksum)
+    return !Checksum;
+  return *RootFile.Checksum == *Checksum;
+}
+
 Expected<unsigned>
 MCDwarfLineTableHeader::tryGetFile(StringRef &Directory,
                                    StringRef &FileName,
@@ -561,6 +570,8 @@ MCDwarfLineTableHeader::tryGetFile(StringRef &Directory,
     trackMD5Usage(Checksum);
     HasSource = (Source != None);
   }
+  if (isRootFile(RootFile, Directory, FileName, Checksum) && DwarfVersion >= 5)
+    return 0;
   if (FileNumber == 0) {
     // File numbers start with 1 and/or after any file numbers
     // allocated by inline-assembler .file directives.
diff --git a/test/MC/ARM/dwarf-asm-multiple-sections.s b/test/MC/ARM/dwarf-asm-multiple-sections.s
index cff8f000731..39c69c9f7d2 100644
--- a/test/MC/ARM/dwarf-asm-multiple-sections.s
+++ b/test/MC/ARM/dwarf-asm-multiple-sections.s
@@ -2,9 +2,9 @@
 // RUN: llvm-dwarfdump -v %t | FileCheck -check-prefix DWARF -check-prefix DWARF45 %s
 // RUN: llvm-dwarfdump --debug-line %t | FileCheck -check-prefix DWARF-DL -check-prefix DWARF-DL-5 -DDWVER=5 -DDWFILE=0 %s
 // RUN: llvm-objdump -r %t | FileCheck -check-prefix RELOC -check-prefix RELOC5 %s
-// RUN: llvm-mc < %s -triple=armv7-linux-gnueabi -filetype=obj -o %t -g -fdebug-compilation-dir=/tmp
+// RUN: llvm-mc < %s -triple=armv7-linux-gnueabi -filetype=obj -o %t -g -dwarf-version 4 -fdebug-compilation-dir=/tmp
 // RUN: llvm-dwarfdump -v %t | FileCheck -check-prefix DWARF -check-prefix DWARF45 %s
-// RUN: llvm-dwarfdump --debug-line %t | FileCheck -check-prefix DWARF-DL -DDWVER=4 -DDWFILE=1 %s
+// RUN: llvm-dwarfdump --debug-line %t | FileCheck -check-prefix DWARF-DL -check-prefix DWARF-DL-4 -DDWVER=4 -DDWFILE=1 %s
 // RUN: llvm-objdump -r %t | FileCheck -check-prefix RELOC -check-prefix RELOC4 %s
 // RUN: llvm-mc < %s -triple=armv7-linux-gnueabi -filetype=obj -o %t -g -dwarf-version 3 -fdebug-compilation-dir=/tmp
 // RUN: llvm-dwarfdump -v %t | FileCheck -check-prefix DWARF -check-prefix DWARF3 %s
@@ -57,11 +57,14 @@ b:
 // DWARF-DL-5:    include_directories[  0] = "/tmp"
 // DWARF-DL:      file_names[  [[DWFILE]]]:
 // DWARF-DL:      name: "{{(<stdin>|-)}}"
-// DWARF-DL:      0x0000000000000000     17      0      1   0   0  is_stmt
-// DWARF-DL-NEXT: 0x0000000000000004     17      0      1   0   0  is_stmt end_sequence
-// DWARF-DL-NEXT: 0x0000000000000000     21      0      1   0   0  is_stmt
-// DWARF-DL-NEXT: 0x0000000000000004     21      0      1   0   0  is_stmt end_sequence
-
+// DWARF-DL-5:      0x0000000000000000     17      0      0   0   0  is_stmt
+// DWARF-DL-5-NEXT: 0x0000000000000004     17      0      0   0   0  is_stmt end_sequence
+// DWARF-DL-5-NEXT: 0x0000000000000000     21      0      0   0   0  is_stmt
+// DWARF-DL-5-NEXT: 0x0000000000000004     21      0      0   0   0  is_stmt end_sequence
+// DWARF-DL-4:      0x0000000000000000     17      0      1   0   0  is_stmt
+// DWARF-DL-4-NEXT: 0x0000000000000004     17      0      1   0   0  is_stmt end_sequence
+// DWARF-DL-4-NEXT: 0x0000000000000000     21      0      1   0   0  is_stmt
+// DWARF-DL-4-NEXT: 0x0000000000000004     21      0      1   0   0  is_stmt end_sequence
 
 // DWARF: .debug_ranges contents:
 // DWARF: 00000000 ffffffff 00000000
diff --git a/test/MC/ELF/debug-mixed-md5.ll b/test/MC/ELF/debug-mixed-md5.ll
index 2ec8141325f..d48e42c8d12 100644
--- a/test/MC/ELF/debug-mixed-md5.ll
+++ b/test/MC/ELF/debug-mixed-md5.ll
@@ -1,8 +1,7 @@
 ; RUN: %llc_dwarf -filetype=asm -dwarf-version=5 %s -o - | FileCheck %s -check-prefix=ASM
 ; RUN: %llc_dwarf -filetype=obj -dwarf-version=5 %s -o - | llvm-dwarfdump -debug-line - | FileCheck %s -check-prefix=OBJ
 ; ASM: .file 0 "{{.+}}" md5
-; ASM: .file 1 "{{.+}}" md5
-; ASM: .file 2 "t1.cpp"
+; ASM: .file 1 "t1.cpp"
 ; ASM-NOT: md5
 ; OBJ: file_names[ 0]:
 ; OBJ-NOT: md5
diff --git a/test/MC/ELF/dwarf-file0.s b/test/MC/ELF/dwarf-file0.s
index 1a3afb6875f..f98fdcc2b40 100644
--- a/test/MC/ELF/dwarf-file0.s
+++ b/test/MC/ELF/dwarf-file0.s
@@ -19,16 +19,14 @@
 # CHECK:       file_names[ 1]:
 # CHECK-NEXT:  name: "header.h"
 # CHECK-NEXT:  dir_index: 1
-# CHECK:       file_names[ 2]:
-# CHECK-NEXT:  name: "root.cpp"
+# CHECK-4:     file_names[ 2]:
+# CHECK-4-NEXT: name: "root.cpp"
 # CHECK-4-NEXT: dir_index: 2
-# CHECK-5-NEXT: dir_index: 0
 
 # ASM-NOT: .file
 # ASM-5:   .file 0 "/test" "root.cpp"
 # ASM:     .file 1 "/include" "header.h"
 # ASM-4:   .file 2 "/test" "root.cpp"
-# ASM-5:   .file 2 "root.cpp"
 # ASM-NOT: .file
 
 # WARN:      file 0 not supported prior to DWARF-5
diff --git a/test/tools/llvm-objdump/Inputs/embedded-source b/test/tools/llvm-objdump/Inputs/embedded-source
index 072b5a90c5c83aadeb6396d0523f57962d6f586e..f59027e0859e4d62becf5338984eb2f72f1fb043 100644
GIT binary patch
literal 9448
zcmeHMeQX>@6@R<yJKME$zQhf28%VY^ZXHpd?;OXz^mET2$r_wCO`Q@b>3Z*OZ6Dl+
zx!Vg))k2GDshb2638aDq4dRaqq(c8}g;Wx`YC!zcAR!^qN(B)Vl@u8Yl?62Bz1ex^
zyS+0-Lh4^0Y4&~0`@Q+tnb|k9Um2U&uPO>~QsHhO>6~T=mG=bb`KS=3E)K0=z|HV+
z2%@G5vmr`|dK}Xg>6&Saw9}9hF0OitxM~Q7DObmoTSzn@Ecszb#-lydAWTdFbRiRt
zij0sDkxmJQ=@nVQvZRco<`M3Y#2u12rst$RQ|=%2jq;@I-=u1wM6y8ybn#rHWJ~GL
zWj9Uyfb8Tzz#uis9=%OS08ANoKjLK9rNw_IVOe}w*0<icoR|G$%EQx_%_Ij0`m*WX
zY$jix@12hi_6`n2N`*+j=r{Q%+XMF=5jF8R>O%6$Xb#BX5n8gZ&eHUMB5i23rGe#c
z)aecMfbCCFa|gk)h__0vU39ISoyo(2iSgkPt3T2o*^8w!xk6r+TIk7Zv4Sn7w(>ZZ
zZ_Kn(*~esE%N<jKqjy4foK~vz)qOV?uC|yb15UdR=9A}Lb@g>~;nkKUh`lquc<Ett
z@ek(0AD0hJj$b^l<4iCwUU;#I$i>HztZuypVSn$~ISTMnPk0K<yXfh5zC^W>yVjqR
zV)d=*GfTp1I?}Q8YobZ72>U1gBFKennz^`aUbu3Psk~-hy5?@KcxaX${P=e}cKq}4
zyWVWm0m_|6%*D%B9$8&oeJ$`S)F{W6>gpr=KR>fpKS3^z{hQjd_}GfMSYEz3c9jO^
z<k;1P^UC7bRb-AWO>=t_pH4iOSX@mUNlZ?h?*4MC4oA$>y+j`vUwm(T@i!B@FN?9f
z@V<8C`D?4I=9BNZdhB<MpICf*V)4CE^ssvU8|K22V(xph{5FmCL&p-IOB_o)oUoS8
zthMjmB^*I|H{xy3yHr8huHDgtO{BG##VT;vVFG*YB7GF;w~$^%`UcYLF;h|=ISk5t
zSlQm*+;T=~4wIhtHXZSwAOo2i+k3WT|2o>=O16nm_$hT{Q!t4RQKUB9iTN!ocaXwx
zDEv)zIMnqGZ7igpZW<2toC=ts=u<&66kljQ7&3~Xcp?-{gnEWUI@+QAaHvJhXPOx;
z*#7GINiH|i#t3YTz{Ut{jKIbSY>dFh2+#=3Vf;zSkd$LxrtP8>;Azbe+$JgSlINKl
zq|E2A?NX+ghSH550aKp4C<da$<rITanpX`X@Q-V&1)^tEgGlA9xS|>)inAQi7gd7@
zTqmA*4waP75tO<-0$U|z(kz*yvOnC8sMO;jKqSUvI40T?m-ZM|37N-dS;|paF5w><
z;dAT<Rd`@z<aWJh)JbOSydLlEkH+HB{#d*>wp))!qLG-+8okGvpL2?toRfF$>~8%b
zw^+`9K`%PTGo?%+ulMgA8j24cgH+be&*;aUqR{R`A5pz$)^+DfxA*nUWZc<uGLkCf
z`m)*M-lKP}Sopk?awAx%@99tPi|*@>C1V47WARvgz)7WIgM-OEL-s)bo~V;_hNkxo
z#`f)o6!t6tig-^7v`Y?^s!&lZtpYuj2Q^S8k<rw%^N5h_To9C_sBKcuW?MidEd-hZ
zO~GTx1d}8+2k|VpS?TWX3~1fOZQ*U(yGz?PZ{HF`*Aaynqf6_)z3q;+TW=Weya%Ve
z764VM#yipXK#dsK1zPvrZEZLggB?HyU{gIf>4%Uiit?|}CL%P6Fe9^*SkRQ6YDZv)
zA{Lqb)@Pr4|Cu*NzrQ?led2|omvN=Es&rhWqy&Q3wcHeFDhFE62Gro4DxPYT59^gU
zxula$JL%q1p<GNkkrec~xw$@P-p<Ws9S}$GOx^{%IFq8saY)VDMaTfYK5rPf8$Cj%
z4gt(!egyN^F`vSG2J^Que;4y}m}}~&Zvt9;P{R3v_f<xnnT9iHvY}q9>`T=k4Ns1|
zKh^33m3^sJAB2^L{+prEzM<hn;r*u8Pb=`gRja?QvOm}A+bZ{fwfc78{kvAb38Dd`
zS|xZ5LY4cg>Ou`VV5*^gxN<*NYrnbj9jMiB!TY(XYL(zM=mfrpsV>y$RF3x{{-oh<
z-_=n65up1Kf6`!^UnOL%^7W`K)_}iuem#HX{CfTx`t|%3^y~Td$*<p0tB{fgJL~kc
zk2JwwSN#RF@>FPp*EOT2h^|FR4{~GS!keVt@ae?{25aLf_ARvCk6D5G_nVV`IL<&(
zq!aj5@V0{E2C0T;k!jTa_4@T=;im(31r2slp^f~!Buc||^YE9bSMUbXKM#K`?fvb%
zEcEs3Vnz6=pPwP@z!n^5y8WZXcQgA@uQ!;tJ5V1+y?@?z2|xAgw_E6i>zc=?^y6O_
z_oH5~vGjx~)NfX|g6>-{Inu-0{=jkDqo}8T(RC&zad(CERm@?WCsCiDPfGi9nt>Jo
ze0O#V^<?kw@3YdLzZka)6u|dTPwn*EKPBuN?1!nMTXM_O)A+SYJ4I(EQ*xc6<>stZ
zwvcy9cu$-zSTor|(#~3Gw@@ruc6lB!g27Lk>!c$?dt!9e+`!11&g3(eT`bxsEDV~8
zCt$j0=Nv0t&gD)Zq)M{T(ycMA|BzYM{=<obW7gQcqn3s68b}(f(a+qQI5<91TOuyM
zQH9s_)|e?hn4^cmIxunHaALx`Z~y)W#wM-F#PGxzc@<aYsZv=q9xNwqyEd!WZqf~-
z_9~sMysLDyt+`jnktr3dSv#M`2*4V@53SOfyj3na*yOd^s52-hOC{+>1Phimd2mE_
z7$T(;IoD1ib&DRIWe$T1r#J_Ze8F`hiQ(~H*PemMOuihMwM(-QNuS6g*rRUIE0Gau
zjbLF}(aG8bfQV>Q#C7Ifh|sIVvVs_nh%+l+-fS8XEO`j;EqVy%(Vd;kq_D06`oc$K
zE{lxk4Eqce5mR7|Iwx74!FM}dVBSt9G00{K|H?SbWA~ZO*{K>#ni}T$LMy8<Jza8K
ztjAN46CCHFTr>X<p)VOOt;)U?R}GZb{{Q*EXk=TRZbqRQGw*vGZ}I;%z1kRg-d|X~
z7a97c#`t;FK#9(cf>gfPWJ5_D4v?~+81nDoS&i`=znzf^ZoeN-z9`*+na-Ase^E71
zqI0MqV0r|CNU85!f0xvcAk{E4enmA<0*ng<shnRu$@9TH(NAHfvnS&@Zj6#sOjIu=
zZa>>GJ&I*?mSvga%echr1jgiVzo!M5L59YX@f?R9mH4PM<of;n|00&txNtl9KH+JJ
z_eWy<-lSXjk`K?{k8=|LRo{dQGf>jUe0Yw3cgY6xxu0SF`WGag|50+B9F=&#J<Ci_
zN<5Dr{}1N>#J6$9k}{6f-|^u&?w*qM|AGxA<?(0r50Rl*PYzcl?&AN(etVXg{uHqt
zRd~M7G39@CwnGz<SZ4Y&)Y7}>`Z><x|MKVL`{Vh+IDU_>B9=@T&*y?=Y7{26^Q5))
z$9}x{ZVg{KKS&_2)AjHa2Uge5@l=<@uU9|&`z<om7H&VsEj<#*?PPo3bcQ#vfOJtd
zkd*N+`}B-w`X?-BzS{KnvVL0M)yZ%BH!Sq)$ll!Sq3F+gb*i&PMV$@-&r7^h4MyH?
z+b;3`{<q5o`Ga-v>}Q+A`^?}4iDx@5_sh4ega3)d%VA`AUHkjj!M`H${`K##e@x>2
z{eN5H{U<@jS5w}5Dvd-Y-04YOa^c;CB7G})KJvPt{gLOchwz>~@<{w6u1FU8L_9k*
P#$S?0=OaFVPlW#fs%%1*

literal 9936
zcmeHNTWlOx89uY)T|05Iwv&cB3Di?s6Nkp*wc~6y_gP<(F?E`dI3=h!<K3NEd!xOW
z-PvFVqEeYgvO>TSAQfIfpo#|`0143`1W5=qLPb^bfRLb-LPSt#Q*hM~MU-aw|1;;`
zogJ@9#RC$;M>_jozyDn4%sFTF*}>txK1pJfd~7#EX-?L-%K2sN>W8=}bP=|UY3y<K
z7z=>L4_V_Dqz*^)o(N5}LHNl)>mWybjw0gII79S<j+zmr8Y0E=DoGQpLw90@AQ5Fu
z;WFaTqzAbObc{1ZuZse12}(Gu9^%FXZcN~a&Ix~_)IMw*<`L1p5uXMU${G@;@ZSnW
zUrYyY+9vV?qLBlB4b>n!bQ6~tBTBfB15Pwuc>H_PEX;>Qd5iVSdC@+i)IDA4R6N|<
zl{P!mscdnob1D+<4EL&qoZ7?Njq%ZU-&03;Nz{)DmmC|~DV|V2kTinFsv2H6J(U`I
z2K7u}wZhCWkT(i;EN>f`SSkye@k}l&GzQx@Jk-~3^r$`R7Kr2woCGz*I#PcX*QJ)B
zjrxDSX^fYOeeZ6|J(pfg0q&>I+-}g%_)o1+nEw1l+gG}(&-}9CGPmJR_gh1=*N*A4
zH}si5%^w^Yx^giDBd=e%bg>S}l{p}#HDQ3?X=)o_?0i@A7(1eOb^y_5?gjPPn<v(v
z$4D5yW%L3XO4pN3ZC^ko8@=4rrm>_-^3TzJM1StDJiw*<G6c|P=k-gs_UO{f`nCJ^
znsT&8iq_NyaUdj=|M^{iY*ZL4wj2TAt!$}O8hzRSDyXD)cu5zI!ULnv>|sU!n_mH3
zIoR6>C|w=>%Spgg>Vw)Vga5>;&Yrrb&lcye4Bo~LJag*y%oS<&)a^5aw|U`Cd!nC+
zJ{_GcMUO;BhR?R2Sf)T@&UPX_JT!ZEX!f_m8*lPHzx1Ab>)HFIlK%W#w$l9vYX9);
z8^g1A2OvV}q1W`8%aXqB_2L^iCeMsUKNTH~9*Y{6=PLDk2ZoBr6Qd@<T#(kwJ363>
zb&!Xl2<R3z46U+(eh%nYf!+rC8qkLzW0FoDV$xKzw0=c>!<<y#jC#J-0saZ@;r!J0
zcYM+RD)?_e*J!Z$w6A|9tpQkvosh4C-9md`u=&frzF_N@<iVhFwyrPO@kPHL44n?>
z!N^Sg{-Bl*Mxwz`G}zG>RKO4X`+^O;zSz?ZP=EJ4#*j;CsRx#NV5tX|dSIysmU`g-
zy9dT#Uy4Z+lnd}fdt#z1gke(FIJZ*Bz2bbcR>*WNTQ6ifSFIEBLD*Je;(I2bWdHm9
zQV#V~J`G9vta#n0f#jU!kiOv4kg(<a7tW!A;`stot3%itL5b80Micd?azjE7hXbAn
z58E$ZA4T}XHj2yCAM-+P6?OstuMs-OKB&Nb{r%gOjwA76)-Ebxb+a1k?CvRYv1g;B
zP<le$TSL8}FiWIk*>UBBl`o`nS!JslQbS6|gl$h2ws&<E5;@ye;nZkac`cnjkx_H`
z@viZd-Q~cnSdW#ABqQ54Z`~F#&25pMEv6X{C6dYBaPQ_V-CH+D!eKL#*!X@zVtgf$
z2!~7vWp0a^y)i4<9ZAL$Taxi`Jk*;sLkV+RC=%Kn4sB!)LpM0P+)o4d5y<?Uh@F#x
z>}L`mZ;7oC94LKwB*TQ;@G3!iT9%|^pp|{+ru=1gG9XED(AN3Rr5hw4Ixv5ozb-Hi
zOdyR?eE?3KEmC{?YQNl`f4KSK_3c0ETiYV73P2GmSXKqJR=It9<BrA+Yll|vfu~0H
zGoMfmt%l&eRigiKCb#cyY=r3@XkzHVl2{NtH@ksKlJstHB@&#Mkb!CAQ?yQM^ELS&
zc}{DS9tRVfzi@R+@XYtV`-5H2+<9r;?_UX{a#^|AlD~d=!=rwG(ZBqhUk>c_!Rbi)
zpPdd?CT^LgWp);F#eBk26Rc|@m$AAE&g|{(vaN#M<?(W+Oe$-$SbjW#KPOmXB9><<
z#@e8>D)S3R9{q60xd%|VRKsC$(ocZA0{L%{??Ao>`8~)!7@B&>@257N+CdwjNCl^L
z2Qpa4O$wH-d=#tt%;-ktv&$6)Z7F4!(z62v<%wNNH*$H)F6OgJYRB?Zv~ogy7D+P9
zq=-*jV3nZ{Mr!Ki^1AH?$#AlMp{Bo|U9YJRuv+V>%<37f!`1N{m?mp(73XER<)rtN
z+bF|ZW3HxuBcu0Nwf_pHiT7Kzer5SyuUa2ulQsRD%J-Jl{>_Zuht>KPhSx9NB*P6_
zt#w~!s~O#AxiK={0>CwqH_7mJY^|yP0K@AfZ<6`?>ea&1Resx6+hj(&0k58R0bV`r
z7QA}eC3y9&OKq1)W{<d}3M=#Hw;~fKGlkePg{Ko{?c1fYLNK`hk=Vj<!It`9US0Ie
zD{K+!nRv;FFiaJC?|uvKeI!=tf2;?l!;mGo#q`dTlc2}=w0}ciOs64BY~g%+5%jg%
zwOD&!;&Fm363~FFq+bDBs~^y#R{U!m+O(klH+Y<d^Z8BTPy0wN!q0uqEPU=-paD{i
zdA<SktDt}I78aAUdNXE#Ua2wvw}QSI^xpa3&EqUwA9iy+4_);$D&lybuj8OssyrQG
z8uTr`HB9k5cc)Pg+Z9@W>IC*{pvQJy6!VJT5rXY?$jvZ+LY_DY;ZN({6C%!wphtgi
zdw)p&BK`*9|6|Z&J-z<l<o-3*+eF?j*u`WLUR;LpvHnBG@X+BA!(gy`w8m2f+sYeu
z#z>@dS*rjZX3iK-=i;%nVcNNT!H5;7SR$91Ok1{Ps#~{&yK6fc$y7FF#Pa#rv|(lK
z{4`7EV;RdZi<!(cKwOdmo_3Y%!gs7;>^&6SKWGd-H2`&WDo%VA++@bUC!dP$AL<7$
zmxX_&02O>cF$Q%JO&>VKjD5og`l7?efxUYV4~`fk(Z1n9EP;R9NEC{^A<VGMm>nZ^
z<&%e|xj%A1cY&Al{R4cfK8Ap;kTWJ?Su<@hW9R^QnW?N%ELc$GN@-YSkmH4d2qQkM
zjO_0hjc00MI%CJ;K<&IkCy0YxhLxXWYBpzEYP4^t(~gZZbv#>CCt`&Orkc}P06Wyq
zI~H-Qs1gjY<*jrK0Zip}Qf+I>W-1OV*mAr(RBJ*E<b(+bk{pCHq7H)i;|Xwsn6XSM
z0R`qDApEG4X<(d)&~T>01k1pr0i6{ujvJH3I3Qp;3OSWc=7^8Q<9X`@NwAN!h=)EQ
zHWN#@F%Zw5o~*<JPa~I17A(751O|cu$cv}!KPULUfk&~t&e8sjzSDZ&r0DBTkAYAR
znbt|#=h1gi#qA6{tw*Hp2IelDi3vaD)3|9$D5!kG(fOM@!OsrJaL6f_Py5Cf1YQ@8
zxW&f06TVgG`+>@ksr>6c4I~EpOis%81dimyARg(HknxO9c-n`yVo*o{hm_k*engLg
z9q$ZCR%DGM6oFT`<Qx&)^%rV34h;4U;WdHR1fK3Gy!}V*$8#^nrFzo-cT(W-gzQcf
zhjgC-#*L@<#hk!@8;!0+-&IL<3K%z@_S3yMV36p3#gP`ypBXS9p1u*&druR1Z+%H7
zdPd->|LD6reUINn1q(_zQh(ipr+xgSDE~L)C@9T8Qhx^+{HuZJuJV4LzR!F8NhbPz
zz&5$?wBI`@{=Yze;KCEhM1Kle>^~}>&JRZ)fOn!Xb`cnyJA|im2K|3xPK>|kj`H(I
z^WfKjb(c@)k$G$sB=U2l%JWP0e-#XFeEHlWfHcn+!~f2Mr~Ox}z%N!l#l!s*)`i+n
z`<xB|q<WIS=nfLu4d750Dr+2u@PGE`2~YG*uoLfg4YoKEuJ7*T^?etNULE>x6nI=8
z+{x>U*AiYG`hP><$J}7xy}oM}!T&?xk3RtasK9&s&lC&7GEadd65S;5o?=*LwWA?F
zvU}wni{P^Y{}=%U_2Qpg1b<H8z3Yz`e@x)L?f;p;Kf6fz6bC7-eQs9`t^hENi5J6%
rf$xP(^O>G2T%T#~16xkt5=Zu5P(i}t5#g*-8y^=h{;UV!5!t^0S1-jP

diff --git a/test/tools/llvm-objdump/X86/function-sections-line-numbers.s b/test/tools/llvm-objdump/X86/function-sections-line-numbers.s
index b932a5d3f32..46607440cb9 100644
--- a/test/tools/llvm-objdump/X86/function-sections-line-numbers.s
+++ b/test/tools/llvm-objdump/X86/function-sections-line-numbers.s
@@ -30,8 +30,7 @@
 _Z2f1v:                                 # @_Z2f1v
 .Lfunc_begin0:
 	.file	0 "/home/avl" "test.cpp" md5 0xefae234cc05b45384d782316d3a5d338
-	.file	1 "test.cpp" md5 0xefae234cc05b45384d782316d3a5d338
-	.loc	1 1 0                   # test.cpp:1:0
+	.loc	0 1 0                   # test.cpp:1:0
 	.cfi_startproc
 # %bb.0:                                # %entry
 	pushq	%rbp
@@ -40,7 +39,7 @@ _Z2f1v:                                 # @_Z2f1v
 	movq	%rsp, %rbp
 	.cfi_def_cfa_register %rbp
 .Ltmp0:
-	.loc	1 1 12 prologue_end     # test.cpp:1:12
+	.loc	0 1 12 prologue_end     # test.cpp:1:12
 	popq	%rbp
 	.cfi_def_cfa %rsp, 8
 	retq
@@ -55,7 +54,7 @@ _Z2f1v:                                 # @_Z2f1v
 	.type	_Z2f2v,@function
 _Z2f2v:                                 # @_Z2f2v
 .Lfunc_begin1:
-	.loc	1 2 0                   # test.cpp:2:0
+	.loc	0 2 0                   # test.cpp:2:0
 	.cfi_startproc
 # %bb.0:                                # %entry
 	pushq	%rbp
@@ -64,7 +63,7 @@ _Z2f2v:                                 # @_Z2f2v
 	movq	%rsp, %rbp
 	.cfi_def_cfa_register %rbp
 .Ltmp2:
-	.loc	1 2 12 prologue_end     # test.cpp:2:12
+	.loc	0 2 12 prologue_end     # test.cpp:2:12
 	popq	%rbp
 	.cfi_def_cfa %rsp, 8
 	retq
-- 
2.50.1