================
@@ -373,7 +373,6 @@ void yaml::MappingContextTraits<MemoryDescriptor,
yaml::BinaryRef>::mapping(
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);
----------------
labath wrote:
I actually think we should provide a way to override the DataSize field, as the
main purpose of the yaml is to test the parser with possibly invalid input. It
doesn't have to be present by default though, which I think you can achieve by
using `IO.mapOptional("Data Size", Memory.DataSize, Content.binary_size());` --
and putting it after the "Content" line.
(I'm not using hex, because the RawContentStream is also printed in decimal,
but I could be convinced to change both).
https://github.com/llvm/llvm-project/pull/101272
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
