[Lldb-commits] [PATCH] D145624: [lldb] Make MemoryCache::Read more resilient
Alex Langford via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Mar 13 15:38:47 PDT 2023
bulbazord added inline comments.
================
Comment at: lldb/source/Target/Memory.cpp:245-246
+ if (m_invalid_ranges.FindEntryThatContains(cache_line_base_addr)) {
+ error.SetErrorStringWithFormat("memory read failed for 0x%" PRIx64,
+ cache_line_base_addr);
+ return dst_len - bytes_left;
----------------
clayborg wrote:
> Is this an error here? We already got something from the first read and we are just returning partial data, do we need an error? If we fail the first read, then this is an error.
If the second cache line you read is in an invalid range, maybe the user would want some feedback about why it was a partial read. It's a detectable condition. Maybe we shouldn't set an error string though, idk.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145624/new/
https://reviews.llvm.org/D145624
More information about the lldb-commits
mailing list