[PATCH] D140439: [BOLT][DWARF] Change rangelists to use DW_RLE_offset_pair

[Maksim Panchenko via Phabricator via llvm-commits]

maksfb accepted this revision.
maksfb added a comment.
This revision is now accepted and ready to land.

Could you add to the summary the decrease in binary/DWARF size you see with this change?



================
Comment at: bolt/lib/Core/DebugData.cpp:215
+    return false;
+  uint64_t Base = Ranges[Index].LowPC;
+  std::vector<std::pair<uint16_t, uint16_t>> RangeOffsets;
----------------



================
Comment at: bolt/lib/Core/DebugData.cpp:238
+                         support::little);
+  uint32_t BaseIndex = AddrWriter.getIndexFromAddress(Base, CU);
+  encodeULEB128(BaseIndex, OS);
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140439/new/

https://reviews.llvm.org/D140439