aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/MC/MCDwarf.cpp
diff options
context:
space:
mode:
authorFangrui Song <maskray@google.com>2020-03-03 13:58:02 -0800
committerFangrui Song <maskray@google.com>2020-03-03 16:20:40 -0800
commit90acc505edf46410a44f45ff01eb301de21936fb (patch)
tree204374e4f5faff95d924aa6a4a17baa8e4f07e5d /llvm/lib/MC/MCDwarf.cpp
parent31e03317633909c50ead53edf8a19b60698075cc (diff)
downloadllvm-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.cpp32
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);