[Lldb-commits] [lldb] [lldb] Clear error in Target::ReadMemory before calling Process::ReadMemory (PR #217148)

[Felipe de Azevedo Piovezan via lldb-commits]

felipepiovezan wrote:

> It's a little ugly that a few of the Process classes override ReadMemory and not DoReadMemory, so you had to set this is a handful of places. I don't see why you need to do this in Target::ReadMemory now, but otherwise this looks ok.

Is it too late for us to make only _one_ of these methods virtual?

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