[llvm] [BOLT][DWARF] Refactor legacy ranges writers (PR #96006)

Sayhaan Siddiqui via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 16:17:37 PDT 2024


================
@@ -177,6 +177,12 @@ uint64_t DebugRangesSectionWriter::getSectionOffset() {
   return SectionOffset;
 }
 
+void DebugRangesSectionWriter::appendToRangeBuffer(
+    const DebugBufferVector &CUBuffer) {
+  *RangesStream << CUBuffer;
+  SectionOffset = RangesBuffer->size();
----------------
sayhaan wrote:

We would still need `appendToRangeBuffer()` since `RangesStream` is a protected member and we call `appendToRangeBuffer()`  outside of `DebugRangesSectionWriter`, correct? But since it's a single line now we can define it in `DebugData.h` instead of `DebugData.cpp`.

https://github.com/llvm/llvm-project/pull/96006


More information about the llvm-commits mailing list