From 783d84bb395ce2549a14ff96db4df5ac3a513d97 Mon Sep 17 00:00:00 2001 From: Ali Tamur Date: Fri, 19 Apr 2019 02:26:56 +0000 Subject: [PATCH] [llvm] Prevent duplicate files in debug line header in dwarf 5: another attempt Another attempt to land the changes in debug line header to prevent duplicate files in Dwarf 5. I rolled back my previous commit because of a mistake in generating the object file in a test. Meanwhile, I addressed some offline comments and changed the implementation; the largest difference is that MCDwarfLineTableHeader does not keep DwarfVersion but gets it as a parameter. I also merged the patch to fix two lld tests that will strt to fail into this patch. Original Commit: https://reviews.llvm.org/D59515 Original Message: 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. llvm-svn: 358732 --- lld/test/ELF/debug-line-obj.s | 13 +++--- lld/test/ELF/debug-line-str.s | 3 +- llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h | 5 +++ llvm/include/llvm/MC/MCDwarf.h | 50 ++++++++++++++------- llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp | 5 ++- llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp | 19 +++++++- llvm/lib/MC/MCAsmStreamer.cpp | 3 +- llvm/lib/MC/MCContext.cpp | 5 ++- llvm/lib/MC/MCDwarf.cpp | 16 ++++++- llvm/test/MC/ARM/dwarf-asm-multiple-sections.s | 16 ++++--- llvm/test/MC/ELF/debug-mixed-md5.ll | 3 +- llvm/test/MC/ELF/dwarf-file0.s | 6 +-- .../test/tools/llvm-objdump/Inputs/embedded-source | Bin 9936 -> 9528 bytes .../X86/function-sections-line-numbers.s | 9 ++-- 14 files changed, 101 insertions(+), 52 deletions(-) diff --git a/lld/test/ELF/debug-line-obj.s b/lld/test/ELF/debug-line-obj.s index 615673b..32182a1 100644 --- a/lld/test/ELF/debug-line-obj.s +++ b/lld/test/ELF/debug-line-obj.s @@ -32,11 +32,10 @@ _Z3barv: .Lfunc_begin0: .file 0 "/path" "test.cpp" md5 0x9ff11a8404ab4d032ee2dd4f5f8c4140 - .file 1 "test.cpp" md5 0x9ff11a8404ab4d032ee2dd4f5f8c4140 - .loc 1 2 0 # test.cpp:2:0 - .loc 1 2 20 prologue_end # test.cpp:2:20 + .loc 0 2 0 # test.cpp:2:0 + .loc 0 2 20 prologue_end # test.cpp:2:20 callq _Z3foov - .loc 1 2 13 is_stmt 0 # test.cpp:2:13 + .loc 0 2 13 is_stmt 0 # test.cpp:2:13 .Lfunc_end0: .size _Z3barv, .Lfunc_end0-_Z3barv # -- End function @@ -45,10 +44,10 @@ _Z3barv: .type _Z3bazv,@function _Z3bazv: # @_Z3bazv .Lfunc_begin1: - .loc 1 3 0 is_stmt 1 # test.cpp:3:0 - .loc 1 3 20 prologue_end # test.cpp:3:20 + .loc 0 3 0 is_stmt 1 # test.cpp:3:0 + .loc 0 3 20 prologue_end # test.cpp:3:20 callq _Z3foov - .loc 1 3 13 is_stmt 0 # test.cpp:3:13 + .loc 0 3 13 is_stmt 0 # test.cpp:3:13 .Lfunc_end1: .size _Z3bazv, .Lfunc_end1-_Z3bazv diff --git a/lld/test/ELF/debug-line-str.s b/lld/test/ELF/debug-line-str.s index e23b5c8..af5eee4 100644 --- a/lld/test/ELF/debug-line-str.s +++ b/lld/test/ELF/debug-line-str.s @@ -27,8 +27,7 @@ main: .Lfunc_begin0: .file 0 "/home/path" "test.cpp" md5 0x8ed32099ab837bd13543fd3e8102739f - .file 1 "test.cpp" md5 0x8ed32099ab837bd13543fd3e8102739f - .loc 1 3 10 prologue_end + .loc 0 3 10 prologue_end jmp _Z3foov .Lfunc_end0: diff --git a/llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h b/llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h index 0fc34a8..1503a92 100644 --- a/llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h +++ b/llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h @@ -243,6 +243,9 @@ public: bool hasFileAtIndex(uint64_t FileIndex) const; /// Extracts filename by its index in filename table in prologue. + /// In Dwarf 4, the files are 1-indexed and the current compilation file + /// name is not represented in the list. In Dwarf 5, the files are + /// 0-indexed and the primary source file has the index 0. /// Returns true on success. bool getFileNameByIndex(uint64_t FileIndex, const char *CompDir, DILineInfoSpecifier::FileLineInfoKind Kind, @@ -275,6 +278,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 diff --git a/llvm/include/llvm/MC/MCDwarf.h b/llvm/include/llvm/MC/MCDwarf.h index 63040cb..2a24347 100644 --- a/llvm/include/llvm/MC/MCDwarf.h +++ b/llvm/include/llvm/MC/MCDwarf.h @@ -41,11 +41,14 @@ class raw_ostream; class SMLoc; class SourceMgr; -/// Instances of this class represent the name of the dwarf -/// .file directive and its associated dwarf file number in the MC file, -/// and MCDwarfFile's are created and uniqued by the MCContext class where -/// the file number for each is its index into the vector of DwarfFiles (note -/// index 0 is not used and not a valid dwarf file number). +/// Instances of this class represent the name of the dwarf .file directive and +/// its associated dwarf file number in the MC file. MCDwarfFile's are created +/// and uniqued by the MCContext class. In Dwarf 4 file numbers start from 1; +/// i.e. the entry with file number 1 is the first element in the vector of +/// DwarfFiles and there is no MCDwarfFile with file number 0. In Dwarf 5 file +/// numbers start from 0, with the MCDwarfFile with file number 0 being the +/// primary source file, and file numbers correspond to their index in the +/// vector. struct MCDwarfFile { // The base name of the file without its directory path. std::string Name; @@ -225,6 +228,7 @@ public: Expected tryGetFile(StringRef &Directory, StringRef &FileName, Optional Checksum, Optional Source, + uint16_t DwarfVersion, unsigned FileNumber = 0); std::pair Emit(MCStreamer *MCOS, MCDwarfLineTableParams Params, @@ -245,6 +249,18 @@ public: return MCDwarfFiles.empty() || (HasAllMD5 == HasAnyMD5); } + void setRootFile(StringRef Directory, StringRef FileName, + Optional Checksum, + Optional Source) { + CompilationDir = Directory; + RootFile.Name = FileName; + RootFile.DirIndex = 0; + RootFile.Checksum = Checksum; + RootFile.Source = Source; + trackMD5Usage(Checksum.hasValue()); + HasSource = Source.hasValue(); + } + private: void emitV2FileDirTables(MCStreamer *MCOS) const; void emitV5FileDirTables(MCStreamer *MCOS, Optional &LineStr) const; @@ -252,24 +268,23 @@ private: class MCDwarfDwoLineTable { MCDwarfLineTableHeader Header; + bool HasSplitLineTable = false; public: void maybeSetRootFile(StringRef Directory, StringRef FileName, - Optional Checksum, Optional Source) { + Optional Checksum, + Optional Source) { if (!Header.RootFile.Name.empty()) return; - Header.CompilationDir = Directory; - Header.RootFile.Name = FileName; - Header.RootFile.DirIndex = 0; - Header.RootFile.Checksum = Checksum; - Header.RootFile.Source = Source; - Header.trackMD5Usage(Checksum.hasValue()); - Header.HasSource = Source.hasValue(); + Header.setRootFile(Directory, FileName, Checksum, Source); } unsigned getFile(StringRef Directory, StringRef FileName, - Optional Checksum, Optional Source) { - return cantFail(Header.tryGetFile(Directory, FileName, Checksum, Source)); + Optional Checksum, uint16_t DwarfVersion, + Optional Source) { + HasSplitLineTable = true; + return cantFail(Header.tryGetFile(Directory, FileName, Checksum, Source, + DwarfVersion)); } void Emit(MCStreamer &MCOS, MCDwarfLineTableParams Params, @@ -291,12 +306,13 @@ public: Expected tryGetFile(StringRef &Directory, StringRef &FileName, Optional Checksum, Optional Source, + uint16_t DwarfVersion, unsigned FileNumber = 0); unsigned getFile(StringRef &Directory, StringRef &FileName, Optional Checksum, Optional Source, - unsigned FileNumber = 0) { + uint16_t DwarfVersion, unsigned FileNumber = 0) { return cantFail(tryGetFile(Directory, FileName, Checksum, Source, - FileNumber)); + DwarfVersion, FileNumber)); } void setRootFile(StringRef Directory, StringRef FileName, diff --git a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp index eec1028..b14eadd 100644 --- a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp +++ b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp @@ -315,7 +315,9 @@ unsigned DwarfTypeUnit::getOrCreateSourceID(const DIFile *File) { addSectionOffset(getUnitDie(), dwarf::DW_AT_stmt_list, 0); } return SplitLineTable->getFile(File->getDirectory(), File->getFilename(), - getMD5AsBytes(File), File->getSource()); + getMD5AsBytes(File), + Asm->OutContext.getDwarfVersion(), + File->getSource()); } void DwarfUnit::addOpAddress(DIELoc &Die, const MCSymbol *Sym) { @@ -397,7 +399,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/llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp b/llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp index 47df83e..84e13ad 100644 --- a/llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp +++ b/llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp @@ -969,14 +969,29 @@ bool DWARFDebugLine::LineTable::lookupAddressRangeImpl( } bool DWARFDebugLine::LineTable::hasFileAtIndex(uint64_t FileIndex) const { + uint16_t DwarfVersion = Prologue.getVersion(); + assert(DwarfVersion != 0 && "LineTable has no dwarf version information"); + if (DwarfVersion >= 5) + return FileIndex < Prologue.FileNames.size(); 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"); + // Unlike previous versions, in Dwarf 5 the file names is 0-indexed. + if (DwarfVersion >= 5) + return Prologue.FileNames[Index]; + else + return Prologue.FileNames[Index - 1]; +} + Optional 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 source = Entry.Source.getAsCString()) return StringRef(*source); return None; @@ -996,7 +1011,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/llvm/lib/MC/MCAsmStreamer.cpp b/llvm/lib/MC/MCAsmStreamer.cpp index ef5c466..350502d 100644 --- a/llvm/lib/MC/MCAsmStreamer.cpp +++ b/llvm/lib/MC/MCAsmStreamer.cpp @@ -1190,7 +1190,8 @@ Expected MCAsmStreamer::tryEmitDwarfFileDirective( MCDwarfLineTable &Table = getContext().getMCDwarfLineTable(CUID); unsigned NumFiles = Table.getMCDwarfFiles().size(); Expected FileNoOrErr = - Table.tryGetFile(Directory, Filename, Checksum, Source, FileNo); + Table.tryGetFile(Directory, Filename, Checksum, Source, + getContext().getDwarfVersion(), FileNo); if (!FileNoOrErr) return FileNoOrErr.takeError(); FileNo = FileNoOrErr.get(); diff --git a/llvm/lib/MC/MCContext.cpp b/llvm/lib/MC/MCContext.cpp index 4d16715..6ca7d3d 100644 --- a/llvm/lib/MC/MCContext.cpp +++ b/llvm/lib/MC/MCContext.cpp @@ -605,7 +605,8 @@ Expected MCContext::getDwarfFile(StringRef Directory, Optional Source, unsigned CUID) { MCDwarfLineTable &Table = MCDwarfLineTablesCUMap[CUID]; - return Table.tryGetFile(Directory, FileName, Checksum, Source, FileNumber); + return Table.tryGetFile(Directory, FileName, Checksum, Source, DwarfVersion, + FileNumber); } /// isValidDwarfFileNumber - takes a dwarf file number and returns true if it @@ -613,7 +614,7 @@ Expected 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/llvm/lib/MC/MCDwarf.cpp b/llvm/lib/MC/MCDwarf.cpp index 262c520..e8edf6b 100644 --- a/llvm/lib/MC/MCDwarf.cpp +++ b/llvm/lib/MC/MCDwarf.cpp @@ -259,7 +259,7 @@ void MCDwarfLineTable::Emit(MCObjectStreamer *MCOS, void MCDwarfDwoLineTable::Emit(MCStreamer &MCOS, MCDwarfLineTableParams Params, MCSection *Section) const { - if (Header.MCDwarfFiles.empty()) + if (!HasSplitLineTable) return; Optional NoLineStr(None); MCOS.SwitchSection(Section); @@ -538,8 +538,17 @@ Expected MCDwarfLineTable::tryGetFile(StringRef &Directory, StringRef &FileName, Optional Checksum, Optional Source, + uint16_t DwarfVersion, unsigned FileNumber) { - return Header.tryGetFile(Directory, FileName, Checksum, Source, FileNumber); + return Header.tryGetFile(Directory, FileName, Checksum, Source, DwarfVersion, + FileNumber); +} + +bool isRootFile(const MCDwarfFile &RootFile, StringRef &Directory, + StringRef &FileName, Optional Checksum) { + if (RootFile.Name.empty() || RootFile.Name != FileName.data()) + return false; + return RootFile.Checksum == Checksum; } Expected @@ -547,6 +556,7 @@ MCDwarfLineTableHeader::tryGetFile(StringRef &Directory, StringRef &FileName, Optional Checksum, Optional Source, + uint16_t DwarfVersion, unsigned FileNumber) { if (Directory == CompilationDir) Directory = ""; @@ -561,6 +571,8 @@ MCDwarfLineTableHeader::tryGetFile(StringRef &Directory, trackMD5Usage(Checksum.hasValue()); 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/llvm/test/MC/ARM/dwarf-asm-multiple-sections.s b/llvm/test/MC/ARM/dwarf-asm-multiple-sections.s index cff8f00..f995958 100644 --- a/llvm/test/MC/ARM/dwarf-asm-multiple-sections.s +++ b/llvm/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,10 +57,14 @@ b: // DWARF-DL-5: include_directories[ 0] = "/tmp" // DWARF-DL: file_names[ [[DWFILE]]]: // DWARF-DL: name: "{{(|-)}}" -// 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: diff --git a/llvm/test/MC/ELF/debug-mixed-md5.ll b/llvm/test/MC/ELF/debug-mixed-md5.ll index 2ec8141..d48e42c 100644 --- a/llvm/test/MC/ELF/debug-mixed-md5.ll +++ b/llvm/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/llvm/test/MC/ELF/dwarf-file0.s b/llvm/test/MC/ELF/dwarf-file0.s index 1a3afb6..f98fdcc 100644 --- a/llvm/test/MC/ELF/dwarf-file0.s +++ b/llvm/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/llvm/test/tools/llvm-objdump/Inputs/embedded-source b/llvm/test/tools/llvm-objdump/Inputs/embedded-source index 072b5a90c5c83aadeb6396d0523f57962d6f586e..8274a6619ed7e867c376451db1d2d5db42aa5f68 100644 GIT binary patch delta 2806 zcmbVOeN0?c7C-mR2aM2RhGBdR6y{9_3ZpU~e9d6O^3AHW=RAc*(LaKI+e^`tr#>8xEwwoG1()tyuYiG~Bb6+*> zZnE(v_nvcpzjMxg_xqjs)rq^8t6NrWI`C3#0T#9kQ#3lHz(q_Lc7T%*`%3Rqw>5K( ziX{OwVKZturMrbma#$#qyucl(TtqKZL;cdLUwT<`ReG&rAG_Dv?kL8zJ9MG6<54yg z`d>Y)%h+{{DEGN+6ru1aoK<}IUbqUEmF>8uR68OBYPiX&@SL)}=B|7CZogxwHQE~O zfT5v%+3BgFxqLd89~wf{7GibSW&6dVS+0Tc`}Mq1IAa`oz4R0$KJQ!F{Ee~nv2p01 z7yIw;JH4*aHyWom*6duIK0u=2-$`L#R|IzfoM`q8gVD{~yzd16ES}Ew8V3qz$CghC zud!%F@OPYqSB3wE3_yZ&`W_U$Ad0>HX}`I$P?+WA-}j zahC?Wjb~c8ysvNR+rFju?}?l@4t-W?Z2Vxma0I`wkFZG`aP%+@R~)*1q`agQ-*^0y zd5(viO{I92m6pxW9Py&tb=>2w3mvuAQZi>Ci^JB0oMq5*?k+#wI8C7n_!u+~V|a z&Cz_6SZy?(>gebk%b0B=9c>*MvuiAw9!a%znPz9Avn`YE>}rpXrP|UF7^RUpfQiXr zP%bqB6#+GI4S36bZ3A{cDK=$gfg+?j;{-NHs9jk}O_qYfVQ|q7ymzp1{N5W6iY2}H+L;dbQ!gG_>)!MAD>Wa! z9i$|*YA)pf8kILK{(rOab0)*gj+mK@nQ57up3jY%(NT!yvokSsA)TF>G$EUwn1Ts> zz~{fKoR14ifgfL9nyFeFXPop$sQ!WK(^QwK{)y`IRR2u%CpqySzQ^!^s(aZormOrm z2W_&ADwS;ss|5H4Q)#nZBF+Va81k#^u&~NtRmgH^zyZJCwvBub`c>@r-)O6VVS#x- z$795K!N7AA+0U)x?i*Vf>h*SfP_6D&pbXB)(?nk;3cqkU4svJ`hLFVR6k3pwN!(P` zzB@^0KwZgGXSrla;n}P+Y~c?JfBM90{GY`{bu92#!B#+nQ?~+&l2~<7+-cysuCKR5 zR3%t%gcbU(P_FOmyujUHy&9JP7AXv>0O-#BX>rrLQ3+)?0l&g|5zhw6JBiT157pNM z)Z;)v4OELTS}7SNLaAxQgT(G4*7ILt2k=-x?HeZ6tN1~?CUchKK&viuT9?|RqOUUG zuM01xXaQbFZ;cwbTe7P*UEnuM_G!G0*k>hc?dc}TzJRkeDxMA49hGnc9LANJT0B|P zX=`h3#V>2-kI&V<$S_=&z)YQPPiHbY{99cH-Stl_c2EX>Dy(^0P;|2B-$4S^!us*# zh%1LMYcvVczbzD|t$6E-UY3A$wOc38ikE5eoqBIzP0@w9CSwdqI`MLSlWR+|TNnu@ z193SoasE$0#aR^Kz=wlPfs|xZlGP;Z#FNAhN_J4P@vE_ygH0~$$vTV?E$Ox9e^vTE zfwNl7W$lj!JgOyK*2>256SAx=+K!P>(mO0yeOM0s?>HNZxvZn4{CSEn?V6PxJ?t&qr1o9(hI`ltI1ui^{;(bY7~alK;<0d$ z{R=n4s_UE#+J}kyAp25ahPRk#NML{i4XW#i#3lt+j+=z}DvN%95&2Z^=Xi-@l!YGH zdJo$vVqcbi9ueCrus0-ji@TSAQfIfpo#|`0143`1W5=qLPb^bfRLb-LPSt#Q*hM~MU-aw|1;;` zogJ@9#RC$;M>_jozyDn4%sFTF*}>txK1pJfd~7#EX-?L-%K2sN>W8=}bP=|UY3yL4_V_Dqz*^)o(N5}LHNl)>mWybjw0gII79Sn!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_-^3TzJM1StDJiw*|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@ zb&!Xl2!< 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(UBBl`o`nS!JslQbS6|gl$h2ws&}L`mZ;7oC94LKwB*TQ;@G3!iT9%|^pp|{+ru=1gG9XED(AN3Rr5hw4Ixv5ozb-Hi zOdyR?eE?3KEmC{?YQNl`f4KSK_3c0ETiYV73P2GmSXKqJR=It9c_!Rbi) zpPdd?CT^LgWp);F#eBk26Rc|@m$AAE&g|{(vaN#M<< 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{IC*{pvQJy6!VJT5rXY?$jvZ+LY_DY;ZN({6C%!wphtgi zdw)p&BK`*9|6|Z&J-z@dS*rjZX3iK-=i;%nVcNNT!H5;7SR$91Ok1{Ps#~{&yK6fc$y7FF#Pa#rv|(lK z{4`7EV;RdZi^&6SKWGd-H2`&WDo%VA++@bUC!dP$AL<7$ zmxX_&02O>cF$Q%JO&>VKjD5og`l7?efxUYV4~`fk(Z1n9EP;R9NEC{^AnZ^ 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*E01k1pr0i6{ujvJH3I3Qp;3OSWc=7^8Q<9X`@NwAN!h=)EQ zHWN#@F%Zw5o~*@ksr>6c4I~EpOis%81dimyARg(HknxO9c-n`yVo*o{hm_k*engLg z9q$ZCR%DGM6oFT`|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`x6nI=8 z+{x>U*AiYG`hP><$J}7xy}oM}!T&?xk3RtasK9&s&lC&7GEadd65S;5o?=*LwWA?F zvU}wni{P^Y{}=%U_2Qpg1bG2T%T#~16xkt5=Zu5P(i}t5#g*-8y^=h{;UV!5!t^0S1-jP diff --git a/llvm/test/tools/llvm-objdump/X86/function-sections-line-numbers.s b/llvm/test/tools/llvm-objdump/X86/function-sections-line-numbers.s index b932a5d3..4660744 100644 --- a/llvm/test/tools/llvm-objdump/X86/function-sections-line-numbers.s +++ b/llvm/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.7.4