[llvm] [MC][debug_frame] Fix a bug in MCDwarfFrameEmitter::emit() so that per-function CIE can be generated when CIEs are different (PR #192727)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 17 20:35:14 PDT 2026
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp -- llvm/unittests/MC/DwarfDebugFrameCIE.cpp llvm/lib/MC/MCDwarf.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/unittests/MC/DwarfDebugFrameCIE.cpp b/llvm/unittests/MC/DwarfDebugFrameCIE.cpp
index d57e336ec..b3460193c 100644
--- a/llvm/unittests/MC/DwarfDebugFrameCIE.cpp
+++ b/llvm/unittests/MC/DwarfDebugFrameCIE.cpp
@@ -77,13 +77,11 @@ public:
StreamerContext Res;
Res.Ctx = std::make_unique<MCContext>(TT, MAI.get(), MRI.get(),
/*MSTI=*/nullptr);
- Res.MOFI.reset(
- TheTarget->createMCObjectFileInfo(*Res.Ctx, /*PIC=*/false));
+ Res.MOFI.reset(TheTarget->createMCObjectFileInfo(*Res.Ctx, /*PIC=*/false));
Res.Ctx->setObjectFileInfo(Res.MOFI.get());
Res.MII.reset(TheTarget->createMCInstrInfo());
- MCCodeEmitter *MCE =
- TheTarget->createMCCodeEmitter(*Res.MII, *Res.Ctx);
+ MCCodeEmitter *MCE = TheTarget->createMCCodeEmitter(*Res.MII, *Res.Ctx);
MCAsmBackend *MAB =
TheTarget->createMCAsmBackend(*STI, *MRI, MCTargetOptions());
std::unique_ptr<MCObjectWriter> OW = MAB->createObjectWriter(OS);
@@ -138,8 +136,8 @@ TEST_F(DwarfDebugFrameCIE, DistinctReturnColumnsGetDistinctCIEs) {
C.Streamer->finish();
// Parse the emitted ELF and find .debug_frame.
- std::unique_ptr<MemoryBuffer> MB =
- MemoryBuffer::getMemBuffer(ObjContents.str(), "", /*RequiresNullTerminator=*/false);
+ std::unique_ptr<MemoryBuffer> MB = MemoryBuffer::getMemBuffer(
+ ObjContents.str(), "", /*RequiresNullTerminator=*/false);
auto BinOrErr = llvm::object::createBinary(MB->getMemBufferRef());
ASSERT_TRUE(static_cast<bool>(BinOrErr));
auto *ELF = dyn_cast<llvm::object::ELFObjectFileBase>(&**BinOrErr);
``````````
</details>
https://github.com/llvm/llvm-project/pull/192727
More information about the llvm-commits
mailing list