[lldb] [llvm] [Obj2Yaml] Add support for minidump generation with 64b memory ranges. (PR #101272)

Jacob Lalonde via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 30 18:05:54 PDT 2024


================
@@ -356,6 +370,13 @@ void yaml::MappingContextTraits<MemoryDescriptor, yaml::BinaryRef>::mapping(
   IO.mapRequired("Content", Content);
 }
 
+void yaml::MappingContextTraits<MemoryDescriptor_64, yaml::BinaryRef>::mapping(
+    IO &IO, MemoryDescriptor_64 &Memory, BinaryRef &Content) {
+  mapRequiredHex(IO, "Start of Memory Range", Memory.StartOfMemoryRange);
+  mapRequiredHex(IO, "Data Size", Memory.DataSize);
----------------
Jlalond wrote:

Would like feedback from reviewers, should we require DataSize here or should we just take the content size?

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


More information about the llvm-commits mailing list