diff options
author | Fangrui Song <maskray@google.com> | 2020-03-03 13:58:02 -0800 |
---|---|---|
committer | Fangrui Song <maskray@google.com> | 2020-03-03 16:20:40 -0800 |
commit | 90acc505edf46410a44f45ff01eb301de21936fb (patch) | |
tree | 204374e4f5faff95d924aa6a4a17baa8e4f07e5d /llvm/lib/MC/MCDwarf.cpp | |
parent | 31e03317633909c50ead53edf8a19b60698075cc (diff) | |
download | llvm-90acc505edf46410a44f45ff01eb301de21936fb.zip llvm-90acc505edf46410a44f45ff01eb301de21936fb.tar.gz llvm-90acc505edf46410a44f45ff01eb301de21936fb.tar.bz2 |
[MCDwarf] Change emitListsTableHeaderStart to use a reference and fold Start/End symbols generation into it
Apply @dblaikie's suggestions in a post-commit review for D75375
Reviewed By: dblaikie
Differential Revision: https://reviews.llvm.org/D75568
Diffstat (limited to 'llvm/lib/MC/MCDwarf.cpp')
-rw-r--r-- | llvm/lib/MC/MCDwarf.cpp | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/llvm/lib/MC/MCDwarf.cpp b/llvm/lib/MC/MCDwarf.cpp index a0d4756..3c51020 100644 --- a/llvm/lib/MC/MCDwarf.cpp +++ b/llvm/lib/MC/MCDwarf.cpp @@ -45,17 +45,21 @@ using namespace llvm; -void mcdwarf::emitListsTableHeaderStart(MCStreamer *S, MCSymbol *TableStart, - MCSymbol *TableEnd) { - S->AddComment("Length"); - S->emitAbsoluteSymbolDiff(TableEnd, TableStart, 4); - S->emitLabel(TableStart); - S->AddComment("Version"); - S->emitInt16(S->getContext().getDwarfVersion()); - S->AddComment("Address size"); - S->emitInt8(S->getContext().getAsmInfo()->getCodePointerSize()); - S->AddComment("Segment selector size"); - S->emitInt8(0); +MCSymbol *mcdwarf::emitListsTableHeaderStart(MCStreamer &S) { + MCSymbol *Start = + S.getContext().createTempSymbol("debug_list_header_start", true, true); + MCSymbol *End = + S.getContext().createTempSymbol("debug_list_header_end", true, true); + S.AddComment("Length"); + S.emitAbsoluteSymbolDiff(End, Start, 4); + S.emitLabel(Start); + S.AddComment("Version"); + S.emitInt16(S.getContext().getDwarfVersion()); + S.AddComment("Address size"); + S.emitInt8(S.getContext().getAsmInfo()->getCodePointerSize()); + S.AddComment("Segment selector size"); + S.emitInt8(0); + return End; } /// Manage the .debug_line_str section contents, if we use it. @@ -1117,11 +1121,7 @@ static MCSymbol *emitGenDwarfRanges(MCStreamer *MCOS) { if (MCOS->getContext().getDwarfVersion() >= 5) { MCOS->SwitchSection(context.getObjectFileInfo()->getDwarfRnglistsSection()); - MCSymbol *StartSymbol = - context.createTempSymbol("debug_rnglists_start", true, true); - MCSymbol *EndSymbol = - context.createTempSymbol("debug_rnglists_end", true, true); - mcdwarf::emitListsTableHeaderStart(MCOS, StartSymbol, EndSymbol); + MCSymbol *EndSymbol = mcdwarf::emitListsTableHeaderStart(*MCOS); MCOS->AddComment("Offset entry count"); MCOS->emitInt32(0); RangesSymbol = context.createTempSymbol("debug_rnglist0_start", true, true); |