================
@@ -614,6 +614,8 @@ class Process : public
std::enable_shared_from_this<Process>,
return error;
}
+ virtual void DidLoadCore() {}
----------------
medismailben wrote:
I agree with @clayborg: attaching to a process and loading a core file are
mutually exclusive actions so I can use re-using `DidAttach` in core file
process plugins. However, if we rename `DidAttach` to `DidAttachOrLoadCore`
then that would be called for non-corefile process plugins and I think that
would be confusing to that a "LoadCore" mention in the context of the regular
process.
@JDevlieghere if you think that using `DidAttach` in the core file process
plugin context is confusing, I'm fine going back to my original implementation
using `DidLoadCore`.
https://github.com/llvm/llvm-project/pull/70351
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits