[Lldb-commits] [PATCH] D37651: Fix for bug 34532 - A few rough corners related to post-mortem debugging (core/minidump)

[Stephane Sezer via Phabricator via lldb-commits]

sas requested changes to this revision.
sas added a comment.
This revision now requires changes to proceed.

I think it would be cleaner/smaller if you didn't use pure-virtual functions, but instead you had `CanReseume()` return `false` in `Process.h`, and had a default implementation of `DoResume()`  that called `llvm::report_fatal_error()`.

Instead, here, you're defaulting `CanResume()` to `true` and forcing every child class to redefine as `false` and provide an error'ing implementation of `DoResume()`.


https://reviews.llvm.org/D37651