[llvm] [BOLT] Remove workaround for flushPendingLabels (PR #97632)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 3 13:28:44 PDT 2024
https://github.com/MaskRay created https://github.com/llvm/llvm-project/pull/97632
The code emits an empty MCDataFragment to ensure that the labels are
attached to `SplitSection`. The workaround, due to the removed
`flushPendingLabels` mechanism (see
75006466296ed4b0f845cbbec4bf77c21de43b40), is now unneeded.
>From 8e81eb891f3b87967d8f27fda1e0f55930497e22 Mon Sep 17 00:00:00 2001
From: Fangrui Song <i at maskray.me>
Date: Wed, 3 Jul 2024 13:28:35 -0700
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?UTF-8?q?l=20version?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Created using spr 1.3.5-bogner
---
bolt/lib/Core/BinaryContext.cpp | 8 --------
1 file changed, 8 deletions(-)
diff --git a/bolt/lib/Core/BinaryContext.cpp b/bolt/lib/Core/BinaryContext.cpp
index f28a0cd6eb9c6..0a1f1bb9e0d20 100644
--- a/bolt/lib/Core/BinaryContext.cpp
+++ b/bolt/lib/Core/BinaryContext.cpp
@@ -2403,16 +2403,8 @@ BinaryContext::calculateEmittedSize(BinaryFunction &BF, bool FixBranches) {
Streamer->emitLabel(SplitStartLabel);
emitFunctionBody(*Streamer, BF, FF, /*EmitCodeOnly=*/true);
Streamer->emitLabel(SplitEndLabel);
- // To avoid calling MCObjectStreamer::flushPendingLabels() which is
- // private
- Streamer->emitBytes(StringRef(""));
- Streamer->switchSection(Section);
}
- // To avoid calling MCObjectStreamer::flushPendingLabels() which is private or
- // MCStreamer::Finish(), which does more than we want
- Streamer->emitBytes(StringRef(""));
-
MCAssembler &Assembler =
static_cast<MCObjectStreamer *>(Streamer.get())->getAssembler();
Assembler.layout();
More information about the llvm-commits
mailing list