[llvm] [BOLT][DWARF][NFC] Update Die to not use std::optional (PR #97844)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 5 11:02:50 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-bolt
Author: Sayhaan Siddiqui (sayhaan)
<details>
<summary>Changes</summary>
Updates initialization to remove unnecessary use of std::optional.
---
Full diff: https://github.com/llvm/llvm-project/pull/97844.diff
1 Files Affected:
- (modified) bolt/lib/Rewrite/DWARFRewriter.cpp (+4-5)
``````````diff
diff --git a/bolt/lib/Rewrite/DWARFRewriter.cpp b/bolt/lib/Rewrite/DWARFRewriter.cpp
index 1f426d0adfc61..330865e9733bb 100644
--- a/bolt/lib/Rewrite/DWARFRewriter.cpp
+++ b/bolt/lib/Rewrite/DWARFRewriter.cpp
@@ -1638,14 +1638,13 @@ 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);
+ DIEValue DvalGNUBase = 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();
``````````
</details>
https://github.com/llvm/llvm-project/pull/97844
More information about the llvm-commits
mailing list