[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