[llvm] [BOLT][DWARF][NFC] Update Die to not use std::optional (PR #97844)
Sayhaan Siddiqui via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 5 10:54:54 PDT 2024
https://github.com/sayhaan updated https://github.com/llvm/llvm-project/pull/97844
>From 1f54f8698f41e77af22039b9392880a7a4e72112 Mon Sep 17 00:00:00 2001
From: Sayhaan Siddiqui <sayhaan at meta.com>
Date: Fri, 5 Jul 2024 09:34:25 -0700
Subject: [PATCH 1/2] Update Die to not use std::optional
Summary:
Test Plan:
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: https://phabricator.intern.facebook.com/D59399659
---
bolt/lib/Rewrite/DWARFRewriter.cpp | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/bolt/lib/Rewrite/DWARFRewriter.cpp b/bolt/lib/Rewrite/DWARFRewriter.cpp
index 1f426d0adfc61..74286145c5eb7 100644
--- a/bolt/lib/Rewrite/DWARFRewriter.cpp
+++ b/bolt/lib/Rewrite/DWARFRewriter.cpp
@@ -1638,14 +1638,14 @@ void DWARFRewriter::finalizeCompileUnits(DIEBuilder &DIEBlder,
"RangesWriter does not exist for DWOId");
std::unique_ptr<DebugRangesSectionWriter> &LegacyRangesWriter =
RangesWriterIterator->second;
- std::optional<DIE *> Die = LegacyRangesWriter->getDie();
- if (!Die || !Die.value())
+ DIE *Die = LegacyRangesWriter->getDie();
+ if (!Die)
continue;
DIEValue DvalGNUBase =
- Die.value()->findAttribute(dwarf::DW_AT_GNU_ranges_base);
+ Die->findAttribute(dwarf::DW_AT_GNU_ranges_base);
assert(DvalGNUBase && "GNU_ranges_base attribute does not exist for DWOId");
DIEBlder.replaceValue(
- Die.value(), dwarf::DW_AT_GNU_ranges_base, DvalGNUBase.getForm(),
+ Die, dwarf::DW_AT_GNU_ranges_base, DvalGNUBase.getForm(),
DIEInteger(LegacyRangesSectionWriter->getSectionOffset()));
std::unique_ptr<DebugBufferVector> RangesWritersContents =
LegacyRangesWriter->releaseBuffer();
>From c8814fb2cb819812b8ce130b677059004ad1a0f2 Mon Sep 17 00:00:00 2001
From: Sayhaan Siddiqui <sayhaan at meta.com>
Date: Fri, 5 Jul 2024 10:42:28 -0700
Subject: [PATCH 2/2] Formatting changes
---
bolt/lib/Rewrite/DWARFRewriter.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/bolt/lib/Rewrite/DWARFRewriter.cpp b/bolt/lib/Rewrite/DWARFRewriter.cpp
index 74286145c5eb7..330865e9733bb 100644
--- a/bolt/lib/Rewrite/DWARFRewriter.cpp
+++ b/bolt/lib/Rewrite/DWARFRewriter.cpp
@@ -1641,8 +1641,7 @@ void DWARFRewriter::finalizeCompileUnits(DIEBuilder &DIEBlder,
DIE *Die = LegacyRangesWriter->getDie();
if (!Die)
continue;
- DIEValue DvalGNUBase =
- Die->findAttribute(dwarf::DW_AT_GNU_ranges_base);
+ DIEValue DvalGNUBase = Die->findAttribute(dwarf::DW_AT_GNU_ranges_base);
assert(DvalGNUBase && "GNU_ranges_base attribute does not exist for DWOId");
DIEBlder.replaceValue(
Die, dwarf::DW_AT_GNU_ranges_base, DvalGNUBase.getForm(),
More information about the llvm-commits
mailing list