================ @@ -336,3 +336,88 @@ TEST(MinidumpYAML, ExceptionStream_ExtraParameter) { 0xab, 0xad, 0xca, 0xfe}), *ExpectedContext); } + +TEST(MinidumpYAML, MemoryRegion_64bit) { + SmallString<0> Storage; + auto ExpectedFile = toBinary(Storage, R"( +--- !minidump +Streams: + - Type: Memory64List + Memory Ranges: + - Start of Memory Range: 0x7FFFFFCF0818283 + Content: '68656c6c6f' + - Start of Memory Range: 0x7FFFFFFF0818283 + Content: '776f726c64' + )"); + + ASSERT_THAT_EXPECTED(ExpectedFile, Succeeded()); + object::MinidumpFile &File = **ExpectedFile; + + ASSERT_THAT(File.streams().size(), 1u); + + Error Err = Error::success(); + // Explicit Err check + ASSERT_FALSE(Err); ---------------- labath wrote:
This shouldn't be necessary. Could it have something to do with the fact that you're not using the `ErrorAsOutParameter` object inside `getMemory64List`? (That object allows you to assign to an error object without first checking it). 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